* Re: [PATCH] create-spdx: fix config build by adding dependency to enable reruns
[not found] <1673978504-5453-1-git-send-email-maanyagoenka@linux.microsoft.com>
@ 2023-01-17 18:02 ` Maanya Goenka
2023-01-18 10:06 ` [OE-core] " Luca Ceresoli
0 siblings, 1 reply; 2+ messages in thread
From: Maanya Goenka @ 2023-01-17 18:02 UTC (permalink / raw)
To: maanyagoenka@linux.microsoft.com,
openembedded-core@lists.openembedded.org
[-- Attachment #1: Type: text/plain, Size: 2194 bytes --]
Also, signed off by Paul Eggleton: paul.eggleton@microsoft.com
________________________________
From: maanyagoenka@linux.microsoft.com <maanyagoenka@linux.microsoft.com>
Sent: Tuesday, January 17, 2023 10:01 AM
To: openembedded-core@lists.openembedded.org <openembedded-core@lists.openembedded.org>
Cc: Maanya Goenka <maanyagoenka@linux.microsoft.com>; Maanya Goenka <maanyagoenka@microsoft.com>
Subject: [PATCH] create-spdx: fix config build by adding dependency to enable reruns
From: Maanya Goenka <maanyagoenka@linux.microsoft.com>
The issue encountered is about local stos builds failing (when these builds are
preceded by a local SoC OS or MOS build). Essentially, the failure is seen when
building two different configs with shared state, one where gcc-cross-<arch> has a dependency
and one where it doesn't (specifically, one where the abicheck class in meta-binaryaudit
is inherited and one where it isn't). Hence, the task signatures change but a rerun of those said tasks
does not occur. The result is that when the config with the dependency is built and then the one without
is built, due to incorrect dependencies, the SPDX manifest creation stage errors out.
create-spdx relies on BB_TASKDEPDATA to get dependencies and then adds that variable to
vardepsexclude. A change in dependencies therefore, does not result in a
re-execution of the tasks. This commit adds an explicit dependency on DEPENDS which influences
BB_TASKDEPDATA and triggers reruns for new config builds having different dependencies.
Signed-off-by: Maanya Goenka <maanyagoenka@microsoft.com>
---
meta/classes/create-spdx-2.2.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/classes/create-spdx-2.2.bbclass b/meta/classes/create-spdx-2.2.bbclass
index f0513af..e2553b4 100644
--- a/meta/classes/create-spdx-2.2.bbclass
+++ b/meta/classes/create-spdx-2.2.bbclass
@@ -377,7 +377,7 @@ def collect_dep_recipes(d, doc, spdx_recipe):
return dep_recipes
collect_dep_recipes[vardepsexclude] += "BB_TASKDEPDATA"
-
+collect_dep_recipes[vardeps] += "DEPENDS"
def collect_dep_sources(d, dep_recipes):
import oe.sbom
--
1.8.3.1
[-- Attachment #2: Type: text/html, Size: 3239 bytes --]
^ permalink raw reply related [flat|nested] 2+ messages in thread