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

Doesn't work correctly with newer node.js versions #71

Closed
wojciechczerniak opened this issue Dec 8, 2019 · 8 comments
Closed

Doesn't work correctly with newer node.js versions #71

wojciechczerniak opened this issue Dec 8, 2019 · 8 comments
Labels
Bug Something isn't working Chore Devops and scripting. Tasks that are not changing the product

Comments

@wojciechczerniak
Copy link
Contributor

Description

Set the minimal required node.js version in package.json. The requirement comes from GPU.js that requires node >=10.0.0.

@wojciechczerniak wojciechczerniak added the Chore Devops and scripting. Tasks that are not changing the product label Dec 8, 2019
@aninde
Copy link
Contributor

aninde commented Dec 17, 2019

Dependencies do not install correctly and build failed when node v. 12.13.0 is used.

1 error generated.
make[1]: *** [Release/obj.target/webgl/src/bindings.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Users/ndemytruk/Desktop/work/hyper/hyperformula/node_modules/node-gyp/lib/build.js:262:23)
gyp ERR! stack     at ChildProcess.emit (events.js:210:5)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:272:12)
gyp ERR! System Darwin 19.0.0
gyp ERR! command "/Users/ndemytruk/.nvm/versions/node/v12.13.0/bin/node" "/Users/ndemytruk/Desktop/work/hyper/hyperformula/node_modules/gl/node_modules/.bin/node-gyp" "rebuild"
gyp ERR! cwd /Users/ndemytruk/Desktop/work/hyper/hyperformula/node_modules/gl
gyp ERR! node -v v12.13.0
gyp ERR! node-gyp -v v3.8.0

make: *** [setup] Error 1

Build is succesful while using node v11.15.0 (npm v6.7.0).

@wojciechczerniak
Copy link
Contributor Author

Wasn't working for me in v13 either.

@wojciechczerniak wojciechczerniak added the Bug Something isn't working label Dec 17, 2019
@wojciechczerniak wojciechczerniak added this to the January 2020 milestone Dec 27, 2019
@wojciechczerniak wojciechczerniak changed the title Set the minimal required node.js version Not working with newer node.js versions Dec 27, 2019
@wojciechczerniak wojciechczerniak changed the title Not working with newer node.js versions Doesn't work correctly with newer node.js versions Dec 27, 2019
@voodoo11 voodoo11 self-assigned this Feb 21, 2020
This was referenced Feb 25, 2020
@swistak35
Copy link
Contributor

12.13.0 works for me (with cleaning node_modules before npm install), make check afterwards works too
13.9.0 indeed fails, but only because some packages missing on my Ubuntu, found solution here: stackgl/headless-gl#71 (comment) -- installing these packages helped. make check passes.

@wojciechczerniak
Copy link
Contributor Author

I can confirm that the latest develop works with
v13.7 v13.0 v12.16 v11.15

Mac OS X 10.15, no additional packages needed

@voodoo11 voodoo11 assigned aninde and unassigned voodoo11 Feb 29, 2020
@aninde
Copy link
Contributor

aninde commented Mar 2, 2020

I can confirm that branch conformance and develop work on mac OS X 10.15.2 with the latest node 13.9

@wojciechczerniak
Copy link
Contributor Author

Thx

@aninde
Copy link
Contributor

aninde commented Mar 2, 2020

I tried to build HyperFormula on Windows 10 and I failed on node 13.9 and node 10.0.
Can somebody who knows Windows more than me, can try?

I used nvm-windows. After preparing environment (yarn, nvm-windows, node10.0.0 and node13.9.0 installation) I tried to build HF by npm i in repo directory.
Build crashes with the request for installation of prebuild-install. I installed that via npm, and tried npm i once again but it didn't change anything.

I understand that I need to install Python, but this should not happen during npm i?

Here's log with errors rom failder build on Windows 10
2020-03-02T15_29_53_193Z-debug.log

@aninde aninde reopened this Mar 2, 2020
@wojciechczerniak wojciechczerniak assigned voodoo11 and unassigned aninde Mar 2, 2020
@aninde
Copy link
Contributor

aninde commented Mar 3, 2020

I was able to build Hyperformula and run tests on Windows 10 with node 13.9.0 and node 10.14.0, both installed by nvm-windows.
@scarletfog this can be useful for documentation.

Running HyperFormula on Windows 10 required installation of :

There is no Unix make on Windows, so instead of make setup, I enterer yarn install directly.
For testing yarn jest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Chore Devops and scripting. Tasks that are not changing the product
Projects
None yet
Development

No branches or pull requests

4 participants