Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

syntax error in esprima.js? #1451

Closed
loldrup opened this issue Jan 12, 2016 · 6 comments
Closed

syntax error in esprima.js? #1451

loldrup opened this issue Jan 12, 2016 · 6 comments

Comments

@loldrup
Copy link

loldrup commented Jan 12, 2016

How come I get this error:

jon@Lemur:~/frontend$ browserify -t deamdify js/test_browserify.js -o js/test_bundle.js
/home/jon/frontend/node_modules/esprima/esprima.js:3734
throw e;
^

Error: Line 47: Unexpected token =>
at throwError (/home/jon/frontend/node_modules/esprima/esprima.js:1846:21)
at throwUnexpected (/home/jon/frontend/node_modules/esprima/esprima.js:1904:9)
at consumeSemicolon (/home/jon/frontend/node_modules/esprima/esprima.js:1983:13)
at parseStatement (/home/jon/frontend/node_modules/esprima/esprima.js:3281:9)
at parseSourceElement (/home/jon/frontend/node_modules/esprima/esprima.js:3505:20)
at parseFunctionSourceElements (/home/jon/frontend/node_modules/esprima/esprima.js:3334:29)
at parseFunctionExpression (/home/jon/frontend/node_modules/esprima/esprima.js:3477:16)
at parsePrimaryExpression (/home/jon/frontend/node_modules/esprima/esprima.js:2194:24)
at parseLeftHandSideExpressionAllowCall (/home/jon/frontend/node_modules/esprima/esprima.js:2293:61)
at parsePostfixExpression (/home/jon/frontend/node_modules/esprima/esprima.js:2341:16)

@loldrup
Copy link
Author

loldrup commented Jan 12, 2016

This is the content of js/test_browserify.js. Only 'SuperQuadric' is an AMD module :

"use strict";

var qunit = require('qunitjs');
var three = require('three.js'); // remove
var gl_matrix_common = require('gl-matrix-common'); // remove
var gl_matrix_mat3 = require('gl-matrix-mat3'); // remove
var regression_js = require('regression.js'); // remove
var SuperQuadric = require('./cons/designs/SuperQuadric'); // remove

console.log("qunit: ", qunit);
console.log("three: ", three);
console.log("gl_matrix_common: ", gl_matrix_common);
console.log("gl_matrix_mat3: ", gl_matrix_mat3);
console.log("regression-js: ", regression_js);
console.log("SuperQuadric: ", SuperQuadric);

@ariya
Copy link
Contributor

ariya commented Jan 12, 2016

Error: Line 47: Unexpected token =>

Can you find out the content of this Line 47?

It could be also the problem with deamdify.

@loldrup
Copy link
Author

loldrup commented Jan 12, 2016

I now think this line 47 pointer points to my file './cons/designs/SuperQuadric'. At this line, I define an 'arrow' function, which is part of ES6. I only need my project to work in Firefox, so that is OK. I don't need Babel or similar.
But how do I make esprima (or deamdify) accept ES6 syntax?

@ariya
Copy link
Contributor

ariya commented Jan 12, 2016

deamdify needs to use Esprima that supports arrow function. Can you check the Esprima version? Open /home/jon/frontend/node_modules/esprima/esprima.js and search for version.

@loldrup
Copy link
Author

loldrup commented Jan 12, 2016

I found this:

// Sync with *.json manifests.
exports.version = '1.2.5';

@loldrup
Copy link
Author

loldrup commented Jan 13, 2016

I 'transferred' this issue to the deAMDify github account:
jaredhanson/deamdify#36 (comment)

@loldrup loldrup closed this as completed Jan 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants