Switched from browserify to pure-cjs builder

[RReverser]

Switched to recast-based own pure-cjs CommonJS builder which allows to save ~10% of minified+gzipped library size.

Current comparison table is following (used output of grunt-compare-size):

orig	orig.gz	diff	diff.gz	filename	diff %	diff.gz %
391132	77972	12212	-10818	JSXTransformer.js	3%	-14%
538934	110908	-5706	-2861	react-with-addons.js	-1%	-3%
111603	31270	-15423	-3162	react-with-addons.min.js	-14%	-10%
504173	103543	-2489	-2188	react.js	0%	-2%
104632	29316	-14674	-2908	react.min.js	-14%	-10%

[RReverser]

Added some more optimizations + fixes for JSXTransformer build, so providing updated stats:

orig	orig.gz	diff	diff.gz	filename	diff %	diff.gz %
391132	77972	-26812	-20441	JSXTransformer.js	-7%	-26%
538934	110908	-5700	-2859	react-with-addons.js	-1%	-3%
111603	31270	-15423	-3162	react-with-addons.min.js	-14%	-10%
504173	103543	-2483	-2189	react.js	0%	-2%
104632	29316	-14674	-2908	react.min.js	-14%	-10%

[petehunt]

wowzers.

Can you send me a link to the react.js output file so I can take a look at what it looks like?

[RReverser]

Sorry, currently not at machine I have React cloned at. You can get general idea of optimization from https://github.com/RReverser/pure-cjs/blob/master/sample/a.out.js.

Do you want me to clone and build it on this machine?

[benjamn]

This is great!

[petehunt]

So it looks like the name require() never appears in the output bundles, right? This would fix ForbesLindesay/umd#10 for us

[RReverser]

Yep, it doesn't (that's actually the reason I decided to use different name for this function in generated code).
Here are built files: http://sdrv.ms/1dez1Uq

[RReverser]

@petehunt, could you open the link? Any thoughts on this?

[benjamn]

We probably want to change the file names so they don't imply the use of browserify. Maybe something more generic, like grunt/config/bundle.js etc.?

[RReverser]

Yeah, I didn't rename files since it wasn't the most interesting important part so you could firstly have look and make decision before I make final cosmetic changes like this.

[zpao]

Just talked with @benjamn a second too late, but we're going to revert and ship 0.9 with the knowns (browserify) instead of introducing this right now. I want to cut that branch this week so hopefully we'll reland this in master right after that.

[RReverser]

Ok, let me know when you want to merge this again, so we could resolve merge issues if any.

[plievone]

So it looks like the name require() never appears in the output bundles, right?

@petehunt Browserify --standalone since v3.24.0 does this too with derequire.

[vjeux]

Ping, 0.9 is now live :)