Re: [PATCH 3/6] rpm: Enable PACKAGECONFIG[acl audit selinux] by DISTRO_FEATURES

["Jörg Sommer" <joerg.sommer@navimatix.de>]

[-- Attachment #1: Type: text/plain, Size: 1677 bytes --]

Dmitry Baryshkov schrieb am Di 04. Aug, 15:24 (+0300):
> On Tue, Jul 14, 2026 at 04:41:39PM +0200, Jörg Sommer wrote:
> > From: Jörg Sommer <joerg.sommer@navimatix.de>
> >
> > Signed-off-by: Jörg Sommer <joerg.sommer@navimatix.de>
> > ---
> >  meta/recipes-devtools/rpm/rpm_4.20.1.bb | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-devtools/rpm/rpm_4.20.1.bb b/meta/recipes-devtools/rpm/rpm_4.20.1.bb
> > index ba967ec1fa..17e75c3e14 100644
> > --- a/meta/recipes-devtools/rpm/rpm_4.20.1.bb
> > +++ b/meta/recipes-devtools/rpm/rpm_4.20.1.bb
> > @@ -64,7 +64,10 @@ BBCLASSEXTEND = "native nativesdk"
> >  # https://github.com/llvm/llvm-project/issues/82541
> >  TOOLCHAIN = "gcc"
> >
> > -PACKAGECONFIG ??= "archive"
> > +PACKAGECONFIG ??= "\
> > +    ${@bb.utils.filter('DISTRO_FEATURES', 'acl audit selinux', d)} \
> 
> This broke SDK builds for SELinux-enabled distros since nativesdk-rpm
> depends on nativesdk-libselinux, but it is not there by default. [1]
> should fix the issue, if it gets accepted.
> 
> [1] https://lore.kernel.org/r/20260804114749.625810-1-dmitry.baryshkov@oss.qualcomm.com/
> 
> Otherwise, we'd need to drop this patch.

Or we could drop only selinux from the list. But this leaves the question
what someone should do when she enables rpm with selinux and wants an SDK.
So, hopfully your patch gets accepted.


Best regards, Jörg

-- 
Navimatix GmbH           T: 03641 - 327 99 0
Tatzendpromenade 2       F: 03641 - 526 306
07745 Jena               www.navimatix.de

Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480

[-- Attachment #2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 6684 bytes --]