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

Using Linux distributions of node can cause the react build to fail #33

Closed
bassmanitram opened this issue Mar 6, 2017 · 2 comments
Closed

Comments

@bassmanitram
Copy link

bassmanitram commented Mar 6, 2017

This issue isn't specific to this project, but it does cause the project to fail:

> [email protected] build /home/jbartle9/Development/AWS/APIGateway/AIA/aws-api-gateway-developer-portal/dev-portal
> react-scripts build

Creating an optimized production build...
Failed to compile.

static/js/main.91fca694.js from UglifyJs
Unexpected character '`' [/usr/lib/nodejs/events.js:260,0]

This appears to explain the root cause issue (node distribution source making it into NODE_PATH). I am far from proficient in npm/node, let alone react, so I really don't know what needs changing to this project to alleviate this problem for us Linux users.

@bassmanitram bassmanitram changed the title A too-recent version of nope can cause the react build to fail Using Linux distributions of node can cause the react build to fail Mar 6, 2017
@bassmanitram
Copy link
Author

Workaround (at least for Ubuntu 16.04 and Node v6.10 installed from standard repos):

echo $NODE_PATH
NODE_PATH=<previous response with /usr/lib/nodejs: removed> npm run setup

@brettstack
Copy link
Contributor

Thanks for reporting and documenting. I'm going to close this issue since it isn't related to us, but hopefully it will be useful for others who encounter the same issue and come across this via search or other. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants