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

Migrate to GCC 13 #5541

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Migrate to GCC 13 #5541

wants to merge 3 commits into from

Conversation

ktf
Copy link
Member

@ktf ktf commented Jul 5, 2024

Migrate to GCC 13

  • Move to GCC 13.2.0
  • Move to GDB 14.2
  • Do not use GCC 7.3.0 as default, since now AliRoot it's built
    with its own default.
  • Add support for CCACHE. While disabled by default it can be enabled
    by passing CCACHE_DISABLE=false as environment variable.

ktf added 2 commits July 5, 2024 10:02
- Bump to the latest version
- Clean up old workarounds from the recipe
- Move to GCC 13.2.0
- Move to GDB 14.2
- Do not use GCC 7.3.0 as default, since now AliRoot it's built
  with its own default.
- Add support for CCACHE. While disabled by default it can be enabled
  by passing CCACHE_DISABLE=false as environment variable.
@ktf ktf requested a review from a team as a code owner July 5, 2024 08:03
@ktf ktf mentioned this pull request Jul 5, 2024
@davidrohr
Copy link
Contributor

I just checked locally that the Geant header problem we have with new GCC versions is fixed in new Geant versions, i.e. if we have a fix and go back to new Geant reverting #5567, the GCC bump should go through.
@sawenzel : do we any ETA when we can bump Geant again?

@sawenzel
Copy link
Collaborator

@davidrohr : #5574

@davidrohr
Copy link
Contributor

Great, thx!

@davidrohr
Copy link
Contributor

AliceO2Group/AliceO2#13425 and AliceO2Group/AliceO2#13426 should fix O2 compilation with this GCC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants