This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Install GCC 13 in the docker containers
ClosedPublic

Authored by philnik on May 16 2023, 10:45 AM.

Details

Reviewers
ldionne
Mordante
Group Reviewers
Restricted Project
Commits
rG95715ddb5d18: [libc++] Install GCC 13 in the docker containers

Diff Detail

Event Timeline

philnik created this revision.May 16 2023, 10:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 16 2023, 10:45 AM
Herald added a subscriber: arichardson. · View Herald Transcript
philnik requested review of this revision.May 16 2023, 10:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 16 2023, 10:45 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript

I haven't checked whether the container builds yet, since that doesn't work on arm Macs currently.

Mordante accepted this revision as: Mordante.May 16 2023, 11:20 AM
Mordante added a subscriber: Mordante.

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?

ldionne accepted this revision.May 16 2023, 12:08 PM

I don't feel strongly about using jammy+ppa versus lunar, maybe @ldionne does, so I leave the final approval to him.

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:

  1. Build the image
  2. 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 revision is now accepted and ready to land.May 16 2023, 12:08 PM
philnik updated this revision to Diff 523820.May 19 2023, 9:19 AM

Use PPA instead

This revision was landed with ongoing or failed builds.May 19 2023, 9:21 AM
This revision was automatically updated to reflect the committed changes.