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

Luca Bocassi
2021-03-09T11:07:14Z

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

On Mon, 2021-03-08 at 19:29 +0000, Richard Purdie 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
>
> so far.
>
> Cheers,
>
> Richard

Thank you, these seems to be easy fixes: -staticdev package was missing
(forgot it's separate from -dev) and there's a libx32 directory to take
care of. Will do a quick build test and send v8.

--
Kind regards,
Luca Boccassi

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