gulp-jspm-build icon indicating copy to clipboard operation
gulp-jspm-build copied to clipboard

"bundleSfx: true" still returns config.js

Open techniq opened this issue 8 years ago • 5 comments

Currently I'm get "System is not defined" when performing a bundleSfx: true because the config.js is in the pipeline. Unless I'm mistaken, should this not be the case.

Semi-related, it seems like it would be useful to have system.js bundled for buildleSfx: false build (as long as the system.js, config.js, some-bundle.js order was maintained?

techniq avatar Nov 09 '15 18:11 techniq

If SFX, don't output the config, as it isn't necessary?

OliverJAsh avatar Nov 22 '15 22:11 OliverJAsh

Correct, as neither is system.js. It's all self contained.

On Sun, Nov 22, 2015, 5:21 PM Oliver Joseph Ash [email protected] wrote:

If SFX, don't output the config, as it isn't necessary?

— Reply to this email directly or view it on GitHub https://github.com/buddhike/gulp-jspm-build/issues/8#issuecomment-158808461 .

techniq avatar Nov 23 '15 01:11 techniq

@techniq Thanks for getting to the bottom of this. I was wondering why this happens to some users. Bunding system.js for non sfx would be a handy feature (I will no longer have to copy jspm_packages dir to build output then ;-))

BTW: What would happen if you generate many non-fx bundles? Each of them should not have systemjs and configjs right?

buddhike avatar Nov 25 '15 20:11 buddhike

I bumped into this issue recently. This is what I discovered.

  • System not defined error was because my sfx bundle wasn't produced correctly. This was because I has sfx option specified at the wrong level so it was producing a true sfx bundle.
  • config.js is produced in the output although it's not necessary. However having config.js does not cause System is not define error.

Will leave this issue open until we get rid of config.js from the output.

buddhike avatar Feb 09 '16 01:02 buddhike

+1

dwhu avatar Aug 20 '16 17:08 dwhu