diff mbox series

[scarthgap] glibc-testsuite: inherit nospdx to fix do_create_spdx failure

Message ID 20260708115052.71740-1-jaipaul.cheernam@est.tech
State Changes Requested
Delegated to: Yoann Congal
Headers show
Series [scarthgap] glibc-testsuite: inherit nospdx to fix do_create_spdx failure | expand

Commit Message

Jaipaul Cheernam July 8, 2026, 11:50 a.m. UTC
The glibc-testsuite recipe fails during do_create_spdx with:

  ERROR: glibc-testsuite-2.39+git-r1 do_create_spdx: Recipe glibc-testsuite
  is trying to create package sln which was already written by recipe glibc.
  This will cause corruption, please resolve this and only provide the
  package from one recipe or the other or only build one of the recipes.

This happens because glibc-testsuite includes glibc_2.39.bb via 'require'
to reuse its source and build configuration for running 'make check'. As a
result, it inherits all of glibc's PACKAGES definitions (including sln).
While the recipe already inherits 'nopackages' to suppress packaging, the
nopackages class does not disable SPDX tasks. The do_create_spdx task then
attempts to generate SPDX data for packages like 'sln', conflicting with
the real glibc recipe that owns that package.

This issue was exposed by the SPDX 3.0 backport (commit 9c9b954504,
Nov 2025) which introduced stricter package-level SPDX generation.

Fix this by adding 'inherit nospdx'. glibc-testsuite does not run on
target or factor into the build supply chain, since its purpose is to
run tests in QEMU at build time.

(cherry picked from commit 32801348ca231978498612f3ebee121ca27459c1)
Signed-off-by: Jaipaul Cheernam <jaipaul.cheernam@est.tech>
---
 meta/recipes-core/glibc/glibc-testsuite_2.39.bb | 1 +
 1 file changed, 1 insertion(+)

Comments

Yoann Congal July 19, 2026, 5:26 p.m. UTC | #1
On Wed Jul 8, 2026 at 1:50 PM CEST, Jaipaul Cheernam via lists.openembedded.org wrote:
> The glibc-testsuite recipe fails during do_create_spdx with:
>
>   ERROR: glibc-testsuite-2.39+git-r1 do_create_spdx: Recipe glibc-testsuite
>   is trying to create package sln which was already written by recipe glibc.
>   This will cause corruption, please resolve this and only provide the
>   package from one recipe or the other or only build one of the recipes.
>
> This happens because glibc-testsuite includes glibc_2.39.bb via 'require'
> to reuse its source and build configuration for running 'make check'. As a
> result, it inherits all of glibc's PACKAGES definitions (including sln).
> While the recipe already inherits 'nopackages' to suppress packaging, the
> nopackages class does not disable SPDX tasks. The do_create_spdx task then
> attempts to generate SPDX data for packages like 'sln', conflicting with
> the real glibc recipe that owns that package.
>
> This issue was exposed by the SPDX 3.0 backport (commit 9c9b954504,
> Nov 2025) which introduced stricter package-level SPDX generation.
>
> Fix this by adding 'inherit nospdx'. glibc-testsuite does not run on
> target or factor into the build supply chain, since its purpose is to
> run tests in QEMU at build time.
>
> (cherry picked from commit 32801348ca231978498612f3ebee121ca27459c1)

Hello,

That is not a straight forward cherry-pick. Commit message and author
where rewritten.

If you need to add backport info to the cherry-pick, add them after the
"(cherry picked from ..." line but keep the rest (author, title, body)
the same.

You can either send me a v2 with proper cherry-pick procedure (only
append, no rewrite) or just request a cherry-pick for me to handle
(looks easy enough).

Regards,

> Signed-off-by: Jaipaul Cheernam <jaipaul.cheernam@est.tech>
> ---
>  meta/recipes-core/glibc/glibc-testsuite_2.39.bb | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-core/glibc/glibc-testsuite_2.39.bb b/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
> index 2e076f4b0f..e0e3e8ba84 100644
> --- a/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
> +++ b/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
> @@ -31,6 +31,7 @@ do_check:append () {
>  }
>  
>  inherit nopackages
> +inherit nospdx
>  deltask do_stash_locale
>  deltask do_install
>  deltask do_populate_sysroot
Jaipaul Cheernam July 20, 2026, 8:09 a.m. UTC | #2
Hi Yoann,

Please proceed with cherry-pick from your side.

Thanks ..


Regards,
Jaipaul Cheernam

From: Yoann Congal <yoann.congal@smile.fr>
Date: Sunday, 19 July 2026 at 19:26
To: Jaipaul Cheernam <jaipaul.cheernam@est.tech>; openembedded-core@lists.openembedded.org <openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [scarthgap][PATCH] glibc-testsuite: inherit nospdx to fix do_create_spdx failure

On Wed Jul 8, 2026 at 1:50 PM CEST, Jaipaul Cheernam via lists.openembedded.org wrote:
> The glibc-testsuite recipe fails during do_create_spdx with:
>
>   ERROR: glibc-testsuite-2.39+git-r1 do_create_spdx: Recipe glibc-testsuite
>   is trying to create package sln which was already written by recipe glibc.
>   This will cause corruption, please resolve this and only provide the
>   package from one recipe or the other or only build one of the recipes.
>
> This happens because glibc-testsuite includes glibc_2.39.bb via 'require'
> to reuse its source and build configuration for running 'make check'. As a
> result, it inherits all of glibc's PACKAGES definitions (including sln).
> While the recipe already inherits 'nopackages' to suppress packaging, the
> nopackages class does not disable SPDX tasks. The do_create_spdx task then
> attempts to generate SPDX data for packages like 'sln', conflicting with
> the real glibc recipe that owns that package.
>
> This issue was exposed by the SPDX 3.0 backport (commit 9c9b954504,
> Nov 2025) which introduced stricter package-level SPDX generation.
>
> Fix this by adding 'inherit nospdx'. glibc-testsuite does not run on
> target or factor into the build supply chain, since its purpose is to
> run tests in QEMU at build time.
>
> (cherry picked from commit 32801348ca231978498612f3ebee121ca27459c1)

Hello,

That is not a straight forward cherry-pick. Commit message and author
where rewritten.

If you need to add backport info to the cherry-pick, add them after the
"(cherry picked from ..." line but keep the rest (author, title, body)
the same.

You can either send me a v2 with proper cherry-pick procedure (only
append, no rewrite) or just request a cherry-pick for me to handle
(looks easy enough).

Regards,

> Signed-off-by: Jaipaul Cheernam <jaipaul.cheernam@est.tech>
> ---
>  meta/recipes-core/glibc/glibc-testsuite_2.39.bb | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-core/glibc/glibc-testsuite_2.39.bb b/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
> index 2e076f4b0f..e0e3e8ba84 100644
> --- a/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
> +++ b/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
> @@ -31,6 +31,7 @@ do_check:append () {
>  }
>
>  inherit nopackages
> +inherit nospdx
>  deltask do_stash_locale
>  deltask do_install
>  deltask do_populate_sysroot


--
Yoann Congal
Smile ECS
diff mbox series

Patch

diff --git a/meta/recipes-core/glibc/glibc-testsuite_2.39.bb b/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
index 2e076f4b0f..e0e3e8ba84 100644
--- a/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
+++ b/meta/recipes-core/glibc/glibc-testsuite_2.39.bb
@@ -31,6 +31,7 @@  do_check:append () {
 }
 
 inherit nopackages
+inherit nospdx
 deltask do_stash_locale
 deltask do_install
 deltask do_populate_sysroot