diff mbox series

ccache: do not use ccache if it is explicitly forbidden

Message ID 20250807175045.13854-1-peter.marko@siemens.com
State New
Headers show
Series ccache: do not use ccache if it is explicitly forbidden | expand

Commit Message

Marko, Peter Aug. 7, 2025, 5:50 p.m. UTC 
From: Peter Marko <peter.marko@siemens.com>

Some recipes explicitly disable ccache via CCACHE_DISABLE variable since
they are known issues with ccache in those recipes.
This setting should not be ignored also when ccache in in HOSTOOLS.

Rework a hard to read if clause so that it is in format
if (not CCACHE_DISABLE and (cond1 or cond2 or cond2)):

Signed-off-by: Peter Marko <peter.marko@siemens.com>
---
 meta/classes/ccache.bbclass | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/meta/classes/ccache.bbclass b/meta/classes/ccache.bbclass
index 5ce23e50f3..f6bd972ff4 100644
--- a/meta/classes/ccache.bbclass
+++ b/meta/classes/ccache.bbclass
@@ -51,10 +51,10 @@  python() {
     Enable ccache for the recipe
     """
     pn = d.getVar('PN')
-    if ("ccache" in d.getVar("HOSTTOOLS").split() or
+    if (not bb.utils.to_boolean(d.getVar('CCACHE_DISABLE')) and
+        ("ccache" in d.getVar("HOSTTOOLS").split() or
         pn in d.getVar('CCACHE_NATIVE_RECIPES_ALLOWED') or
-        not (bb.data.inherits_class("native", d) or
-        bb.utils.to_boolean(d.getVar('CCACHE_DISABLE')))):
+        not bb.data.inherits_class("native", d))):
         d.appendVar('DEPENDS', ' ccache-native')
         d.setVar('CCACHE', 'ccache ')
 }