fixed #13788 - CI-unixish-docker.yml: added ubuntu:25.10 / cleanups#7472
fixed #13788 - CI-unixish-docker.yml: added ubuntu:25.10 / cleanups#7472firewave merged 1 commit intodanmar:mainfrom
ubuntu:25.10 / cleanups#7472Conversation
|
|
||
| # 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).
There was a problem hiding this comment.
Out of scope of this ticket - I filed https://trac.cppcheck.net/ticket/14490 about it.
firewave
changed the title
ubuntu:24.10 with ùbuntu:25.04 / cleanupsubuntu:24.10 with ubuntu:25.04 / cleanups
firewave
changed the title
ubuntu:24.10 with ubuntu:25.04 / cleanupsubuntu:24.10 with ubuntu:25.10 / cleanups
|
Ubuntu 25.04 is already EOL so move on to 25.10. |
firewave
changed the title
ubuntu:24.10 with ubuntu:25.10 / cleanupsubuntu:25.10 / cleanups
firewave
force-pushed
the
ub2504
branch
2 times, most recently
from
b43d5cc to
deaedf7
Compare
|
2 participants
No description provided.