Re: [OE-core] [PATCH] package: disable renamed dependency error if allarch is overridden

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Tue, 2025-05-20 at 15:06 +0200, Alexander Kanavin via lists.openembedded.org wrote:
> On Mon, 19 May 2025 at 12:24, Oleksandr Hnatiuk via
> lists.openembedded.org <ohnatiuk=cisco.com@lists.openembedded.org>
> wrote:
> > 
> > Also, consider this
> > 
> > packagegroup.bbclass inherits allarch.bbclass - except when PACKAGE_ARCH is set before inheriting packagegroup
> > in order to do so, a nativesdk packagegroup must inherit nativesdk first
> > there is a native-last QA check which strongly suggests inheriting nativesdk last
> > nativesdk is inherited last via inherit_defer in nativesdk-packagegroup-sdk-host
> > 
> > I see all of this as either very misleading at best or a mistake at worst.
> 
> This is probably true. It does look like a pile of ad hoc fixups
> rather than an overall design and structure. Maybe it's possible to
> clean and simplify?
> 
>  I'd say if you resubmit the patch with all the extra research you did
> in the commit message, that would help review. I don't have objections
> anymore.

There is a lot of history here. I'm tempted to suggest we remove the
allarch inherit from packagegroup and add it to the recipes explicitly
as it is all getting far too convoluted and I'd like to see things
simplified, not complicated.

I'd like to see the conditional code in allarch removed too FWIW if we
can work out how to do it.

Cheers,

Richard