RactiveJS and SystemJS 0.11.2 - Module process not declared as a dependency

[victorwpbastos]

After the last update my app stop running:

When I downgrade to version 0.11.1 all back to normal.

[wildlyinaccurate]

Just stumbled across this error as well. Works fine with 0.11.1.

[guybedford]

Thanks, this is due to 3bfddd8.

@theefer it seems like reverting to tokenizing may be the way to go with the rules at this level as this is caused by string characters in regular expressions causing invalid string removal.

For now we should probably revert the change, and then work on a tokenizer for the next release.

[theefer]

Sounds fair. Regexp only get us so far.

Will the tokeniser run in all cases, or do you want to start the work to
pre process deps for all cases except dev?

[guybedford]

The tokenizer will just run in browser in dev, to avoid a full syntax parser. Builds use their own parsing process - https://github.com/systemjs/builder/blob/master/compilers/cjs.js.

[theefer]

OK. Are you thinking of using esprima, or another parser?

[guybedford]

I was thinking simple tokenizing like in

systemjs/loader.js

Line 49 in 5f46011

function removeComments(str) {

.

[theefer]

Ah yes that'd work. Plus I realised today stable esprima doesn't support much of ES6 :-(

[guybedford]

An alternative would be to just use Traceur for CommonJS as well actually, but I'm hesitant.

[guybedford]

Fixed and published the reversion for this as 0.11.3.

[theefer]

Should the bug whose fix broke this and got reverted be reopened?

[guybedford]

@theefer yes, we can reopen the issue.

[theefer]

It's #311 but I don't have the rights to reopen.