Skip to content

Conversation

@avikivity
Copy link
Member

@avikivity avikivity commented Aug 4, 2025

Ubuntu 24.04 has too old compilers (clang 18 and gcc 13); the current ones are clang 21 and gcc 15.

Update the workflow to run on Fedora 43 via a container so we access the most recent compilers.

@avikivity avikivity force-pushed the update-ci-toolchain-f42 branch from 679e40f to 9c37115 Compare August 4, 2025 16:40
@avikivity
Copy link
Member Author

Dependency: #2903

@avikivity avikivity force-pushed the update-ci-toolchain-f42 branch from 9c37115 to 1a2db22 Compare August 7, 2025 09:58
@avikivity
Copy link
Member Author

Rebased past 4ce3e66

@avikivity avikivity force-pushed the update-ci-toolchain-f42 branch from 1a2db22 to c36137a Compare August 9, 2025 17:47
@avikivity avikivity force-pushed the update-ci-toolchain-f42 branch from c36137a to c4ee356 Compare November 21, 2025 15:42
@avikivity avikivity changed the title .github: update test workflow to Fedora 42 .github: update test workflow to Fedora 43 Nov 21, 2025
@avikivity
Copy link
Member Author

v3: rebased past fixes (719d788, b1a7ed5)

Ubuntu 24.04 has too old compilers (clang 18 and gcc 13); the current
ones are clang 21 and gcc 15.

Update the workflow to run on Fedora 43 via a container so we access
the most recent compilers.

Since the basic Fedora container doesn't come with git installed, we
must install it before we checkout submodules.
@avikivity avikivity force-pushed the update-ci-toolchain-f42 branch from c4ee356 to 907cdb1 Compare November 21, 2025 16:00
@avikivity
Copy link
Member Author

v4: rebased past fix (65cbae6)

test:
timeout-minutes: 40
runs-on: ubuntu-24.04
container: fedora:43
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, we should forward ~/.ccache to the container

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or maybe not, if the cache action runs inside the container.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ccache integration appears to be working:

  Cacheable calls:   409 / 409 (100.0%)
    Hits:            408 / 409 (99.76%)
      Direct:        408 / 408 (100.0%)
      Preprocessed:    0 / 408 ( 0.00%)
    Misses:            1 / 409 ( 0.24%)
  Local storage:
    Cache size (GB): 0.0 / 0.5 ( 6.58%)
    Hits:            408 / 409 (99.76%)
    Misses:            1 / 409 ( 0.24%)

One miss from 65cbae6, the rest hits.

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