Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

dockerfile.linux: use Node.js 16.x to avoid glibc issues #265

Merged
merged 1 commit into from
Feb 14, 2023

Conversation

baparham
Copy link
Contributor

@baparham baparham commented Dec 1, 2022

Related to #264 and fixes binary build failures that try to do yum install nodejs @ 18.12.1 since the glibc requirements of that nodejs package are >= 2.28, and it appears the nodejs we need in the environment of the Docker container is used for bootstrapping pkg-fetch to do the actual build

This should work as a workaround for now until a decision is made regarding whether to build on a newer runner for future node.js versions.

@baparham
Copy link
Contributor Author

baparham commented Dec 1, 2022

I've kicked off a Build Node binaries for Linux workflow here to test this out: https://github.com/baparham/pkg-fetch/actions/runs/3597167128

@robertsLando
Copy link
Contributor

cc @jesec

@baparham
Copy link
Contributor Author

baparham commented Feb 1, 2023

@leerob and @robertsLando I've been basing any asset building I do on this commit, which resolves the environment setup issues on the linux builds.

I regard this as a must-merge if we're to make a release anytime soon.

@baparham
Copy link
Contributor Author

baparham commented Feb 1, 2023

an example of a full suite of builds that uses the latest patches and this commit can be found here: https://github.com/baparham/pkg-fetch/actions/runs/4065087301

Copy link
Contributor

@robertsLando robertsLando left a comment

Choose a reason for hiding this comment

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

LGTM for me once the builds succeed

@baparham
Copy link
Contributor Author

baparham commented Feb 1, 2023

https://github.com/baparham/pkg-fetch/actions/runs/4065087301 passed all (except arm) archs with this commit and the fix from #273

@robertsLando
Copy link
Contributor

@leerob 👍🏼

@robertsLando robertsLando changed the title Dockerfile.linux: use Node.js 16.x to avoid glibc issues dockerfile.linux: use Node.js 16.x to avoid glibc issues Feb 14, 2023
@robertsLando robertsLando merged commit 8d896d4 into vercel:main Feb 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants