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

Luca Bocassi
2021-03-09T13:26:59Z

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

On Tue, 2021-03-09 at 12:18 +0100, Köry Maincent wrote:
> 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

Hi,

The DESCRIPTION field is in the util-linux.inc which is included by the
recipe. Or is that something else entirely? What is it, precisely?

--
Kind regards,
Luca Boccassi

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 499 bytes --]