-
Notifications
You must be signed in to change notification settings - Fork 64
INTERNAL COMPILER ERROR. Node(ARRAY_PATTERN) #59
Comments
…FY=false, because Closure Compiler throws an error on our current code; see google/closure-compiler-js#59
* Fix browser build to incorporate Markdown-It, not Marked * Update Google Closure Compiler; recompile browser build but with MINIFY=false, because Closure Compiler throws an error on our current code; see google/closure-compiler-js#59 * Bump version to 2.0.0-beta1; do release build, with updated docs and annotated source
I admit the stack trace is not helpful at all. While I dig into it more, have you tried using the Java version instead (https://www.npmjs.com/package/google-closure-compiler)? |
From commit jashkenas/coffeescript@db216ec, I edited
|
Yeah it's a bug in |
Reduced to a minimal repro: var x;
([x] = [1], 2) |
Hi, sorry for the wait. This particular issue should have been fixed by google/closure-compiler@1ea642a. Could you build closure-compiler-js from HEAD and take a look? Thanks! |
I’m not sure how to do that, but my reproduction instructions above should work for you to test it: git clone https://github.com/jashkenas/coffeescript.git
cd coffeescript
git checkout 9b77371ea846d498b06d0a88aeaa06bafed2abd9
npm install
cake build:browser |
If this is still an issue can you post it at https://github.com/google/closure-compiler? |
I’m a contributor to CoffeeScript and we use closure-compiler-js to minify the browser version of the CoffeeScript compiler. Starting with version 2, CoffeeScript’s compiler is output as ES2015 JavaScript (from original CoffeeScript source) which we then pass to closure-compiler-js.
This worked fine for 2.0.0-alpha1, which started outputting things like classes and default parameters using their ES2015 syntax. In 2.0.0-beta1, which I’m preparing now, we output destructuring using ES2015 syntax; and this seems to throw an error in closure-compiler-js:
You can reproduce this yourself quite easily:
git clone https://github.com/jashkenas/coffeescript.git cd coffeescript git checkout 9b77371ea846d498b06d0a88aeaa06bafed2abd9 npm install cake build:browser
The text was updated successfully, but these errors were encountered: