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

[node][opengl-2] Update workflow for node 21 + allow publish from opengl-2 branch #1941

Merged
merged 13 commits into from
Dec 9, 2023

Conversation

acalcutt
Copy link
Collaborator

@acalcutt acalcutt commented Dec 9, 2023

This PR allows a node release to be published from the 'opengl-2' branch. This would be run by selecting the 'opengl-2' branch when running then workflow
image

Changes

  • added back macos building
  • allow publish steps to work outside main
  • put 'fallback-to-build' back false. I had changed this to true in another PR long ago, but it should be false since the gyp build isn't set up to work.
  • I realized for node 20 support that node-pre-gyp needed the abi list updated. since we are still using my version of that package, I went and updated it to add support for the latest node versions.

Cherry Picks

  • update for node 21 (#1794)
  • Fix flaky Node.js CI with cache (#1443)

@ntadej
Copy link
Collaborator

ntadej commented Dec 9, 2023

I think we should just merge the two Node workflows. Here the the frequency of PRs is so low that we can afford also running Linux and Windows.

Copy link
Collaborator

@ntadej ntadej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

macOS ARM fails due to missing font NotoCJK (we need @louwers to fix it, I think).

Windows build fails due to outdated VCPKG which will come in a separate PR.

@louwers
Copy link
Collaborator

louwers commented Dec 9, 2023

@ntadej I installed it on the Ubuntu ARM runner...

Also installed on macOS now.

@acalcutt
Copy link
Collaborator Author

acalcutt commented Dec 9, 2023

I tried to re-run that macos arm64 job that failed to see if it would pick up the update, but it still failed. I'm not sure it would pick up the new changes yet though.

@ntadej
Copy link
Collaborator

ntadej commented Dec 9, 2023

I'll merge this. We can fix the missing fonts in parallel as I think this has already been failing earlier.

@ntadej ntadej merged commit 53b1b37 into maplibre:opengl-2 Dec 9, 2023
4 of 6 checks passed
acalcutt added a commit to WifiDB/maplibre-native that referenced this pull request Dec 31, 2023
…ngl-2 branch (maplibre#1941)

Co-authored-by: mwilsnd <mwilsnd@gmail.com>
Co-authored-by: Bart Louwers <bart@emeel.net>
acalcutt added a commit to WifiDB/maplibre-native that referenced this pull request Dec 31, 2023
…ngl-2 branch (maplibre#1941)

Co-authored-by: mwilsnd <mwilsnd@gmail.com>
Co-authored-by: Bart Louwers <bart@emeel.net>
acalcutt added a commit to WifiDB/maplibre-native that referenced this pull request Jan 6, 2024
…ngl-2 branch (maplibre#1941)

Co-authored-by: mwilsnd <mwilsnd@gmail.com>
Co-authored-by: Bart Louwers <bart@emeel.net>
acalcutt added a commit to WifiDB/maplibre-native that referenced this pull request Jan 8, 2024
…ngl-2 branch (maplibre#1941)

Co-authored-by: mwilsnd <mwilsnd@gmail.com>
Co-authored-by: Bart Louwers <bart@emeel.net>
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.

None yet

3 participants