diff mbox series

[meta-oe,scarthgap] uutils-coreutils: drop (filtered) "selinux" from PACKAGECONFIG ?=

Message ID 293a5ad3-bf5c-4cec-85d5-4121e01b76b6@gmail.com
State New
Headers show
Series [meta-oe,scarthgap] uutils-coreutils: drop (filtered) "selinux" from PACKAGECONFIG ?= | expand

Commit Message

Clayton Casciato March 26, 2026, 5:24 p.m. UTC
"selinux" adds a dependency on clang-native.

clang-native is provided by meta-clang.

If meta-clang is not included, this surfaces parse-time errors such as:
ERROR: Nothing PROVIDES 'clang-native' (but
[...]/uutils-coreutils_0.0.28.bb DEPENDS on or otherwise requires it)

This is an issue in the default case - where uutils-coreutils is not
enabled.

Suggested-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Clayton Casciato <majortomtosourcecontrol@gmail.com>
---
Sponsor: 21SoftWare LLC

OE-C whinlatter and master include clang-native:
https://git.openembedded.org/openembedded-core/commit/meta/recipes-devtools/clang?h=whinlatter&id=d77e398095228b34851762858a76640e3c2cb0ab

Discussion:
https://lists.openembedded.org/g/openembedded-devel/topic/103440861

 .../recipes-core/uutils-coreutils/uutils-coreutils_0.0.28.bb    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Etienne Cordonnier March 26, 2026, 5:49 p.m. UTC | #1
Hi Clayton,

I agree to merge this patch.

Étienne

On Thu, Mar 26, 2026 at 6:25 PM Clayton Casciato <
majortomtosourcecontrol@gmail.com> wrote:

> "selinux" adds a dependency on clang-native.
>
> clang-native is provided by meta-clang.
>
> If meta-clang is not included, this surfaces parse-time errors such as:
> ERROR: Nothing PROVIDES 'clang-native' (but
> [...]/uutils-coreutils_0.0.28.bb DEPENDS on or otherwise requires it)
>
> This is an issue in the default case - where uutils-coreutils is not
> enabled.
>
> Suggested-by: Khem Raj <raj.khem@gmail.com>
> Signed-off-by: Clayton Casciato <majortomtosourcecontrol@gmail.com>
> ---
> Sponsor: 21SoftWare LLC
>
> OE-C whinlatter and master include clang-native:
>
> https://urldefense.proofpoint.com/v2/url?u=https-3A__git.openembedded.org_openembedded-2Dcore_commit_meta_recipes-2Ddevtools_clang-3Fh-3Dwhinlatter-26id-3Dd77e398095228b34851762858a76640e3c2cb0ab&d=DwICaQ&c=ncDTmphkJTvjIDPh0hpF_4vCHvabgGkICC2epckfdiw&r=AhkbNonVuMIGRfPx_Qj9TsRih1DULJTKUkSGa66m67E&m=4qaMdrtGvkWreWnK7doGMkiFb-BtHVU_UOlvQ23LpOTa-jYEEQoeNRs0MiX43C0B&s=TJh5AjJaNJ4637qYkkSev7d0Jfdmsnw4EdVbI7lfbZA&e=
>
> Discussion:
>
> https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.openembedded.org_g_openembedded-2Ddevel_topic_103440861&d=DwICaQ&c=ncDTmphkJTvjIDPh0hpF_4vCHvabgGkICC2epckfdiw&r=AhkbNonVuMIGRfPx_Qj9TsRih1DULJTKUkSGa66m67E&m=4qaMdrtGvkWreWnK7doGMkiFb-BtHVU_UOlvQ23LpOTa-jYEEQoeNRs0MiX43C0B&s=TzTRDBWSUS36fXqwJJT-NpSGXwehEtgi7MHzNUcEjec&e=
>
>  .../recipes-core/uutils-coreutils/uutils-coreutils_0.0.28.bb    | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-oe/recipes-core/uutils-coreutils/
> uutils-coreutils_0.0.28.bb b/meta-oe/recipes-core/uutils-coreutils/
> uutils-coreutils_0.0.28.bb
> index 76cecac5a6..41e4f1d831 100644
> --- a/meta-oe/recipes-core/uutils-coreutils/uutils-coreutils_0.0.28.bb
> +++ b/meta-oe/recipes-core/uutils-coreutils/uutils-coreutils_0.0.28.bb
> @@ -22,7 +22,7 @@ require ${BPN}-crates.inc
>  PROVIDES = "coreutils"
>  RPROVIDES:${PN} = "coreutils"
>
> -PACKAGECONFIG ?= "${@bb.utils.filter('DISTRO_FEATURES', 'selinux', d)}"
> +PACKAGECONFIG ?= ""
>
>  PACKAGECONFIG[selinux] = "--with-selinux,--without-selinux,libselinux"
>
>
diff mbox series

Patch

diff --git a/meta-oe/recipes-core/uutils-coreutils/uutils-coreutils_0.0.28.bb b/meta-oe/recipes-core/uutils-coreutils/uutils-coreutils_0.0.28.bb
index 76cecac5a6..41e4f1d831 100644
--- a/meta-oe/recipes-core/uutils-coreutils/uutils-coreutils_0.0.28.bb
+++ b/meta-oe/recipes-core/uutils-coreutils/uutils-coreutils_0.0.28.bb
@@ -22,7 +22,7 @@  require ${BPN}-crates.inc
 PROVIDES = "coreutils"
 RPROVIDES:${PN} = "coreutils"
 
-PACKAGECONFIG ?= "${@bb.utils.filter('DISTRO_FEATURES', 'selinux', d)}"
+PACKAGECONFIG ?= ""
 
 PACKAGECONFIG[selinux] = "--with-selinux,--without-selinux,libselinux"