-
Notifications
You must be signed in to change notification settings - Fork 166
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
Missing or unselected GCC 10 on many hosts - Blocks V8 updates! #3317
Comments
https://ci.nodejs.org/job/node-test-commit-custom-suites-freestyle/ runs on the |
This comment was marked as resolved.
This comment was marked as resolved.
@nodejs/build. I pinned this issue. The OP summarises what needs to be updated. It may be a small list, but it includes a large part of our CI hosts. |
Ah this one is because the select-compiler changes we made were to the section where |
Just found that https://ci.nodejs.org/label/ubi81_sharedlibs_openssl111fips_x64/ are still on GCC 8 |
^ @anonrig is looking into it. |
Chromium now officially supports C++20: https://chromium-review.googlesource.com/c/chromium/src/+/4217486 |
Adding build agenda for awareness. This is now blocking further V8 upgrades. |
We need to select the right compiler on @richardlau Can you help with that? |
We already have coverage for Ubuntu 20.04. Refs: nodejs#3317
Opened #3590 |
We already have coverage for Ubuntu 20.04. Refs: #3317
We're close! I think this one just needs another update to https://ci.nodejs.org/job/node-test-commit-arm/50418/nodes=ubuntu2004-armv7l/console |
Yes, I think so. build/ansible/roles/docker/templates/ubuntu2004_armv7l.Dockerfile.j2 Lines 19 to 20 in e2440ff
It looks like currently select-compiler.sh isn't aware of armv7l so it doesn't actually select anything (i.e. leaves things as default) build/jenkins/scripts/select-compiler.sh Lines 19 to 26 in e2440ff
|
Select GCC 10 for Node.js >=20 Refs: nodejs#3317 (comment)
Select GCC 10 for Node.js >=20 Refs: #3317 (comment)
|
@richardlau is this the correct fix, in https://ci.nodejs.org/job/node-test-binary-armv7l/configure: |
@targos Yes, that looks good. |
Done! Resumed build: https://ci.nodejs.org/job/node-test-pull-request/56278/ |
I believe this is done 🎉 |
Easy to spot because GCC 8 and 9 don't support
-std=gnu++20
Latest C++20 CI: https://ci.nodejs.org/job/node-test-pull-request/56278/
Things remaining to update:
The text was updated successfully, but these errors were encountered: