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

Build Error on NodeJS 8 #142

Closed
adl1995 opened this issue Oct 10, 2018 · 4 comments
Closed

Build Error on NodeJS 8 #142

adl1995 opened this issue Oct 10, 2018 · 4 comments
Assignees
Labels
Milestone

Comments

@adl1995
Copy link
Contributor

adl1995 commented Oct 10, 2018

Request Type: Bug

I am running the build on CentOS7 with NodeJS version 8.9.0 (using rh-nodejs8 package). But it seems this version of node does not support node-sass version 3.10.0 as specified in Cortex/www/package.json file. It only supports NodeJS version 0.10, 0.12, 1, 2, 3, 4, 5, and 6.

So, I re-ran the build with NodeJS version 6.11.3 (using rh-nodejs6 package), however, this raises issue with css-spaces dependency, which throws the error:

[error] npm ERR! css-spaces@0.3.5 postinstall: `bower install`
[error] npm ERR! Exit status 1
[error] npm ERR! 
[error] npm ERR! Failed at the css-spaces@0.3.5 postinstall script 'bower install'.
[error] npm ERR! Make sure you have the latest version of node.js and npm installed.
[error] npm ERR! If you do, this is most likely a problem with the css-spaces package,
[error] npm ERR! not with npm itself.

Work Environment

Question Answer
OS version (client) CentOS7
Cortex version / git hash 2.1.0
Package Type From source
@srilumpa
Copy link

What I did as a by-pass to build cortex was to remove the package version dependency in the www/package.json file.

@adl1995
Copy link
Contributor Author

adl1995 commented Oct 10, 2018

@srilumpa - Thanks, that's how I've been able to build it for now.

But, is there a way to test Cortex and see if all the endpoints are working properly, etc.?

@nadouani
Copy link
Contributor

npm install --ignore-scripts should fix the issue, we don't need to have postinstall scripts of dependencies to be executed :)

@nadouani nadouani self-assigned this Oct 17, 2018
@nadouani nadouani added this to the 2.1.3 milestone Oct 17, 2018
@nadouani
Copy link
Contributor

Building with npm install --ignore-scripts works fine

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

No branches or pull requests

3 participants