[OE-core] [PATCH v2] lubusb1: Add udev support

[Richard Purdie]

On Sat, 2019-01-26 at 23:15 -0800, Alistair Francis wrote:
> On Sat, Jan 26, 2019 at 5:11 PM Khem Raj <raj.khem at gmail.com> wrote:
> > On Sat, Jan 26, 2019 at 3:27 PM Richard Purdie
> > <richard.purdie at linuxfoundation.org> wrote:
> > > On Thu, 2019-01-24 at 22:35 +0000, Alistair Francis wrote:
> > > > I don't see circular dependencies anymore between libusb1 and
> > > > udev,
> > > > so
> > > > enable udev support for libusb1.
> > > > 
> > > > Signed-off-by: Alistair Francis <alistair.francis at wdc.com>
> > > > ---
> > > >  meta/recipes-support/libusb/libusb1_1.0.22.bb | 7 ++++---
> > > >  1 file changed, 4 insertions(+), 3 deletions(-)
> > > > 
> > > > diff --git a/meta/recipes-support/libusb/libusb1_1.0.22.bb
> > > > b/meta/recipes-support/libusb/libusb1_1.0.22.bb
> > > > index 0c6e116dd7..35c1d705b6 100644
> > > > --- a/meta/recipes-support/libusb/libusb1_1.0.22.bb
> > > > +++ b/meta/recipes-support/libusb/libusb1_1.0.22.bb
> > > > @@ -20,9 +20,10 @@ S = "${WORKDIR}/libusb-${PV}"
> > > > 
> > > >  inherit autotools pkgconfig ptest
> > > > 
> > > > -# Don't configure udev by default since it will cause a
> > > > circular
> > > > -# dependecy with udev package, which depends on libusb
> > > > -EXTRA_OECONF = "--libdir=${base_libdir} --disable-udev"
> > > > +PACKAGECONFIG ??= "udev"
> > > > +PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev"
> > > > +
> > > > +EXTRA_OECONF = "--libdir=${base_libdir}"
> > > 
> > > Fails in testing with:
> > > 
> > > WARNING: Nothing PROVIDES 'nativesdk-udev' (but
> > > virtual:nativesdk:/home/pokybuild/yocto-worker/a-
> > > quick/build/meta/recipes-support/libusb/libusb1_1.0.22.bb DEPENDS
> > > on or otherwise requires it). Close matches:
> > > WARNING: Nothing PROVIDES 'udev-native' (but
> > > virtual:native:/home/pokybuild/yocto-worker/a-
> > > quick/build/meta/recipes-support/libusb/libusb1_1.0.22.bb DEPENDS
> > > on or otherwise requires it). Close matches:
> > > WARNING: Nothing RPROVIDES 'nativesdk-libusb1-dev' (but
> > > virtual:nativesdk:/home/pokybuild/yocto-worker/a-
> > > quick/build/meta/recipes-support/libusb/libusb1_1.0.22.bb
> > > RDEPENDS on or otherwise requires it)
> > > WARNING: Nothing RPROVIDES 'nativesdk-libusb1' (but
> > > virtual:nativesdk:/home/pokybuild/yocto-worker/a-
> > > quick/build/meta/recipes-support/libusb/libusb1_1.0.22.bb
> > > RDEPENDS on or otherwise requires it)
> > > 
> > 
> > I am seeing these fails too
> > 
> > > Before someone sends a patch creating a nativesdk-udev please
> > > think
> > > very carefully about whether a nativesdk-udev makes sense and
> > > what it
> > > may need to look like.
> 
> What about just disabling the packageconfig for nativesdk? Would that
> be a better option?

Probably easier but what functionality are we missing out on in libusb
and do we need it? This might limit the functionality of tools in the
SDK?

That said, we've been without this so far so you have to wonder how
useful it is even on target...

Cheers,

Richard