From: John Mulligan Date: Thu, 30 Apr 2026 19:56:27 +0000 (-0400) Subject: script: remove call to set ccache option run_second_cpp X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=61e0fcefbb9616d3a1bb932df84996749f705139;p=ceph.git script: remove call to set ccache option run_second_cpp Remove the call to ccache to set run_second_cpp=true because ccache no longer supports this option. See https://ccache.dev/releasenotes.html#_ccache_4_12 for more information. My reading of the ccache release notes & documentation is that when this option was available the default was true. This is what we were setting so removing it should not change behavior for ccache, at least most recent versions. Commit 4cb5a590537a9caaf61db42ce8ea123d2ab961f3 seems to be the source of this line that sets run_second_cpp=true but that change does not clearly say why it is being explicitly set, so I think it's safe to remove. Signed-off-by: John Mulligan --- diff --git a/src/script/run-make.sh b/src/script/run-make.sh index 91fa97ce2451..49f144acc5bb 100755 --- a/src/script/run-make.sh +++ b/src/script/run-make.sh @@ -102,7 +102,6 @@ EOM # remove the entropy generated by the date/time embedded in the build $DRY_RUN export SOURCE_DATE_EPOCH="946684800" $DRY_RUN ccache -o sloppiness=time_macros - $DRY_RUN ccache -o run_second_cpp=true if in_jenkins; then # Build host has plenty of space available, let's use it to keep # various versions of the built objects. This could increase the cache hit