Details
- Reviewers
ldionne Mordante - Group Reviewers
Restricted Project - Commits
- rG95715ddb5d18: [libc++] Install GCC 13 in the docker containers
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
I haven't checked whether the container builds yet, since that doesn't work on arm Macs currently.
Thanks for working on this. I was waiting for GCC-13 to be available in ppa to make this change.
It seems it's there for 3 days https://launchpad.net/~ubuntu-toolchain-r/+archive/ubuntu/test
(As a side note I wanted to update the Clang version in the CI to take advantage of -std=c++26 (landed yesterday), instead I wait for you to do that with this patch.)
I don't feel strongly about using jammy+ppa versus lunar, maybe @ldionne does, so I leave the final approval to him.
libcxx/utils/ci/Dockerfile | ||
---|---|---|
33 | This change may break things, updating to Jammy had some issues. Can you make sure this works with our CI before committing? |
I don't have a preference either, as long as we're running on a stable version of Ubuntu. This is a stable version, right? We don't need LTS or anything, we just don't want something unstable (equivalent of ToT or whatever).
This LGTM but indeed it would be nice to test this before landing:
- Build the image
- Run one or two CI configurations just to make sure it works. Maybe the sanitizers since they sometimes seem more reliant on the OS?
This change may break things, updating to Jammy had some issues. Can you make sure this works with our CI before committing?