fixed #13788 - CI-unixish-docker.yml: replaced ubuntu:24.10 with ubuntu:25.04 / cleanups#7472
fixed #13788 - CI-unixish-docker.yml: replaced ubuntu:24.10 with ubuntu:25.04 / cleanups#7472firewave wants to merge 3 commits intodanmar:mainfrom
ubuntu:24.10 with ubuntu:25.04 / cleanups#7472Conversation
…buntu:25.04` / cleanups
|
|
||
| # TODO: is this actually applied to the guest? | ||
| env: | ||
| # TODO: figure out why there are cache misses with PCH enabled |
There was a problem hiding this comment.
It turns out this was working just fine in this workflow although we forgot to disable the precompiled headers. So we should be able to enable them in the other jobs as well.
There was a problem hiding this comment.
The first run also got cache hits which was unexpected. Turns out we are lacking the branch name in the cache key which causes it to be used across all builds (IIRC I assumed the cache is limited to the branch). So any PR will affect the cache of main builds.
There was a problem hiding this comment.
We also create a new cache with each build because it automatically appends a timestamp. That is done to prevent issues with concurrent access (see hendrikmuhs/ccache-action#117). It seems that might not be necessary for branches but since multiple pushes trigger multiple builds we cannot disable this. That would require us to cancel the already running builds (see https://trac.cppcheck.net/ticket/12325).
firewave
changed the title
ubuntu:24.10 with ùbuntu:25.04 / cleanupsubuntu:24.10 with ubuntu:25.04 / cleanups
1 participant
No description provided.