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

Missing or unselected GCC 10 on many hosts - Blocks V8 updates! #3317

Closed
5 tasks done
targos opened this issue Apr 20, 2023 · 17 comments
Closed
5 tasks done

Missing or unselected GCC 10 on many hosts - Blocks V8 updates! #3317

targos opened this issue Apr 20, 2023 · 17 comments

Comments

@targos
Copy link
Member

targos commented Apr 20, 2023

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:

@targos
Copy link
Member Author

targos commented Apr 25, 2023

https://ci.nodejs.org/job/node-test-commit-custom-suites-freestyle/ runs on the
ubuntu1804-64 label.
One option could be to use an ubuntu2004 or ubuntu2204 label.

@targos

This comment was marked as resolved.

@targos
Copy link
Member Author

targos commented Sep 27, 2023

@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.

@richardlau
Copy link
Member

  • armv7l (despite Ubuntu 20.04)

Ah this one is because the select-compiler changes we made were to the section where "$SELECT_ARCH" = "ARM64" which won't be true for armv7l. I think my preference is to work towards adding x64 Ubuntu 20.04 into the test CI matrix and then move the compiler selection for Ubuntu 20.04 so that it is not architecture specific.

@targos
Copy link
Member Author

targos commented Nov 7, 2023

Just found that https://ci.nodejs.org/label/ubi81_sharedlibs_openssl111fips_x64/ are still on GCC 8

@targos
Copy link
Member Author

targos commented Nov 7, 2023

^ @anonrig is looking into it.

@targos
Copy link
Member Author

targos commented Nov 14, 2023

Chromium now officially supports C++20: https://chromium-review.googlesource.com/c/chromium/src/+/4217486

@targos
Copy link
Member Author

targos commented Nov 29, 2023

Adding build agenda for awareness. This is now blocking further V8 upgrades.

@targos targos changed the title Missing or unselected GCC 10 on many hosts Missing or unselected GCC 10 on many hosts - Blocks V8 updates! Dec 8, 2023
@targos
Copy link
Member Author

targos commented Dec 10, 2023

We need to select the right compiler on ubi81: https://ci.nodejs.org/job/node-test-commit-linux-containered/40782/nodes=ubi81_sharedlibs_openssl111fips_x64/console

@richardlau Can you help with that?

targos added a commit to targos/nodejs-build that referenced this issue Dec 10, 2023
We already have coverage for Ubuntu 20.04.

Refs: nodejs#3317
@richardlau
Copy link
Member

We need to select the right compiler on ubi81: https://ci.nodejs.org/job/node-test-commit-linux-containered/40782/nodes=ubi81_sharedlibs_openssl111fips_x64/console

@richardlau Can you help with that?

Opened #3590

targos added a commit that referenced this issue Dec 11, 2023
We already have coverage for Ubuntu 20.04.

Refs: #3317
@targos
Copy link
Member Author

targos commented Dec 11, 2023

We're close!

I think this one just needs another update to select-compiler.sh?

https://ci.nodejs.org/job/node-test-commit-arm/50418/nodes=ubuntu2004-armv7l/console

@richardlau
Copy link
Member

We're close!

I think this one just needs another update to select-compiler.sh?

https://ci.nodejs.org/job/node-test-commit-arm/50418/nodes=ubuntu2004-armv7l/console

Yes, I think so. gcc-10 should be available


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)
case $NODE_NAME in
*ppc64*le* ) SELECT_ARCH=PPC64LE ;;
*s390x* ) SELECT_ARCH=S390X ;;
*aix* ) SELECT_ARCH=AIXPPC ;;
*x64* ) SELECT_ARCH=X64 ;;
*arm64* ) SELECT_ARCH=ARM64 ;;
*ibmi73* ) SELECT_ARCH=IBMI73 ;;
esac

targos added a commit to targos/nodejs-build that referenced this issue Dec 11, 2023
targos added a commit that referenced this issue Dec 12, 2023
@targos
Copy link
Member Author

targos commented Dec 12, 2023

@targos
Copy link
Member Author

targos commented Dec 13, 2023

@richardlau
Copy link
Member

@targos Yes, that looks good.

@targos
Copy link
Member Author

targos commented Dec 13, 2023

Done!

Resumed build: https://ci.nodejs.org/job/node-test-pull-request/56278/

@targos
Copy link
Member Author

targos commented Dec 22, 2023

I believe this is done 🎉

@targos targos closed this as completed Dec 22, 2023
@targos targos unpinned this issue Dec 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants