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

Fix macOS CI by pinning NODE_VERSION to 12.13.0 #181

Merged
merged 4 commits into from
Jan 7, 2020

Conversation

dhritzkiv
Copy link
Member

@dhritzkiv dhritzkiv commented Jan 1, 2020

This solves macOS CI for the time being, as a quick fix.

There seemed to be 2 issues with macOS builds:

  1. nvm wasn't properly utilizing the NODE_VERSION specified in the test.sh script (despite being set in install.sh), and was defaulting to node 13, causing a NODE_MODULE_VERSION mismatch (72 vs. 79)
  2. Using the latest node 12 released resulted in gl not returning a context instance as no display is found in 12.13.1 and higher (in this case 12.14.0). See Fix no display found error on Node 12.13.1+ on macOS #180

@robertleeplummerjr
Copy link
Collaborator

Does this work for 12.13.1+?

@dhritzkiv
Copy link
Member Author

@robertleeplummerjr not yet. I'm still working on the solution to that.

@dhritzkiv
Copy link
Member Author

@robertleeplummerjr I updated the issue #180 with information about what's causing the breakage in 12.13.1+ and what may happen in the future to fix the issue.

Just thought you'd like to know for your projects

Add section about platform and node version support, including a note about current Node.js breakage on macOS; Other tidying up of the README.
@dhritzkiv dhritzkiv merged commit 9eb9b13 into master Jan 7, 2020
@dhritzkiv dhritzkiv deleted the node-12-ci-fixes_simplified branch January 7, 2020 05:41
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

Successfully merging this pull request may close these issues.

3 participants