Skip to content

Enforce gcc13 for LHAPDF too#191

Merged
tvami merged 1 commit into
mainfrom
FixForV6
May 29, 2026
Merged

Enforce gcc13 for LHAPDF too#191
tvami merged 1 commit into
mainfrom
FixForV6

Conversation

@tvami
Copy link
Copy Markdown
Member

@tvami tvami commented May 28, 2026

Not a 100% how this happened (*) but on trunk the build failed:
https://github.com/LDMX-Software/dev-build-context/actions/runs/26589468131

(*) I think in the PR we prob still had a cached layer for the LHAPDF

In this PR now I enforce LHAPD uses gcc13 too

@tvami tvami requested a review from tomeichlersmith May 29, 2026 15:14
@tvami
Copy link
Copy Markdown
Member Author

tvami commented May 29, 2026

@tomeichlersmith do you have any other idea why this situation happened? If this is just what we have, I'd like to merge this w/o waiting 3 days and test it on v5.0.1

@tomeichlersmith
Copy link
Copy Markdown
Member

I have no idea why it happened, but I'm fine with you making hot-fixes onto main.

I would just delete the old v6.0.0 release and make a new one. The build failing means that there isn't event a v6.0.0 tag on DockerHub for folks to pull from.

@tvami tvami merged commit 46f8f23 into main May 29, 2026
2 of 3 checks passed
@tvami tvami deleted the FixForV6 branch May 29, 2026 16:27
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.

2 participants