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

Update checkout CI action to v4 #216

Closed
wants to merge 22 commits into from

Conversation

jefftrull
Copy link
Collaborator

There are some errors with glibc versions being out of date vs. the run container which this may help.

@jefftrull jefftrull marked this pull request as draft August 26, 2024 04:49
I think this may allow it to run directly in ubuntu 20.04. At any rate
it will match the accumulators library, which passes for these two compilers.
Atomic doesn't seem to need this, maybe wave doesn't either
This may be necessary on Ubuntu 18.04 (our chosen container)
The code for the Ubuntu 18.04 container appears to work in a
separately downloaded Docker image so I'm trying to do comparisons
My theory is that since we run in a container it doesn't matter, plus
Github deprecates these periodically in an unpleasant manner (randomly
shuts them off for brief periods). Furthermore this makes the config
file closer to Boost.Atomic, which works.
@jefftrull jefftrull force-pushed the bugfix/update-ci-checkout branch from 89c9c01 to 5477b3e Compare August 26, 2024 21:15
Previous code (for 16.04 and 18.04) does things that are not needed in 20.04
Many comments online suggest v4 is to blame for glibc version errors
@jefftrull jefftrull force-pushed the bugfix/update-ci-checkout branch 2 times, most recently from 29dbb3d to a0a79cb Compare August 26, 2024 21:56
@jefftrull jefftrull force-pushed the bugfix/update-ci-checkout branch from a0a79cb to ba3df0d Compare August 26, 2024 21:57
@jefftrull jefftrull closed this Aug 27, 2024
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.

1 participant