Re: [OE-core] [PATCH v7] util-linux: split uuid in separate recipe to allow bootstrapping

Kory Maincent
2021-03-09T11:19:04Z

On Mon, 08 Mar 2021 19:29:20 +0000
"Richard Purdie" <richard.purdie@linuxfoundation.org> wrote:

> On Fri, 2021-03-05 at 11:02 +0000, luca.boccassi@gmail.com wrote:
> > From: Luca Boccassi <luca.boccassi@microsoft.com>
> >
> > Recently util-linux gained an (optional) build dependency on libcryptsetup.
> > But libcryptsetup build-depends on util-linux for blkid (optional, can be
> > disabled) and uuid (mandatory).
> > Split out util-linux-uuid in a different recipe to break the cycle.
> >
> > https://github.com/karelzak/util-linux/pull/898
> >
> > Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>
> > ---
> > v1: util-linux 2.35 is not out yet, but I'd like to get the preparatory work
> >     underway as I'm not sure if this is the best approach or if there are
> >     alternatives. Suggestions and comments very welcome. Thanks!
> > v2: changed packages names to reflect old ones (eg: libuuid1 ->
> > util-linux-libuuid) and leave uuid build enable in main recipe to allow for
> > uuidgen build to happen, as it does not have its own autoconf switch.
> > Delete the library manualy from the main recipe after build instead, and
> > add dependency. Might help to break loop python3 -> util-linux ->
> > libselinux -> python3, as it's only libuuid that is needed, see
> > https://lists.yoctoproject.org/g/yocto/message/47570 v3: rebased and
> > refactored to have a common util-linux.inc file v4: added RDEPENDS on
> > util-linux-libuuid on various packages to fix QA warnings v5: remove
> > RDEPENDS and instead have util-linux RDEPEND on util-linux-uuid. Removed
> > PACKAGES_remove and instead filter out libuuid via the package generation
> > regex. Rebased on util-linux 2.36.2.
> > v6: install libuuid.a in libuuid-dev
> >     change rdepends on uuid to use binary package rather than source
> >     add rdepends on uuid-dev to libuuid-dev
> >     remove rprovides of libuuid-dev from uuid-dev
> > v7: do not use '_append' for RDEPENDS, as it is not supported by
> > BBCLASSEXTEND, use simply '+='
>
> This is looking closer but not there yet I'm afraid. Testing showed:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/57/builds/3152
> https://autobuilder.yoctoproject.org/typhoon/#/builders/47/builds/3188
>

It miss also the Maintainer description for linux-util-uuid recipe:
https://autobuilder.yoctoproject.org/typhoon/#/builders/86/builds/1901/steps/14/logs/stdio

Regards,

Köry