[PATCH] libusb: add missing build dependency on udev

[PATCH] libusb: add missing build dependency on udev

[Fabien Chereau]

This fixes the ./configure error in case libusb is built before udev/systemd.

Signed-off-by: Fabien Chereau <fabien.chereau@intel.com>
---
 meta/recipes-support/libusb/libusb1_1.0.19.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-support/libusb/libusb1_1.0.19.bb b/meta/recipes-support/libusb/libusb1_1.0.19.bb
index a20e1fc..be2be9d 100644
--- a/meta/recipes-support/libusb/libusb1_1.0.19.bb
+++ b/meta/recipes-support/libusb/libusb1_1.0.19.bb
@@ -6,6 +6,8 @@ SECTION = "libs"
 LICENSE = "LGPLv2.1+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=fbc093901857fcd118f065f900982c24"
 
+DEPENDS_append = " udev"
+
 BBCLASSEXTEND = "native nativesdk"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/libusb/libusb-${PV}.tar.bz2 \
-- 
1.8.3.2

---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris, 
92196 Meudon Cedex, France
Registration Number:  302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.

Re: [PATCH] libusb: add missing build dependency on udev

[Richard Purdie]

On Thu, 2014-06-26 at 12:20 +0200, Fabien Chereau wrote:
> This fixes the ./configure error in case libusb is built before udev/systemd.
> 
> Signed-off-by: Fabien Chereau <fabien.chereau@intel.com>
> ---
>  meta/recipes-support/libusb/libusb1_1.0.19.bb | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta/recipes-support/libusb/libusb1_1.0.19.bb b/meta/recipes-support/libusb/libusb1_1.0.19.bb
> index a20e1fc..be2be9d 100644
> --- a/meta/recipes-support/libusb/libusb1_1.0.19.bb
> +++ b/meta/recipes-support/libusb/libusb1_1.0.19.bb
> @@ -6,6 +6,8 @@ SECTION = "libs"
>  LICENSE = "LGPLv2.1+"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=fbc093901857fcd118f065f900982c24"
>  
> +DEPENDS_append = " udev"
> +
>  BBCLASSEXTEND = "native nativesdk"
>  
>  SRC_URI = "${SOURCEFORGE_MIRROR}/libusb/libusb-${PV}.tar.bz2 \

This causes the build to explode with circular dependency issues. How
did you test this?

https://autobuilder.yoctoproject.org/main/builders/nightly-ppc-lsb/builds/147/steps/BuildImages/logs/stdio

There is a second error about a missing nativesdk-udev dependency (to
build nativesdk-libusb1) too.

Cheers,

Richard

Re: [PATCH] libusb: add missing build dependency on udev

[Chereau, Fabien]

Hi Richard,

In fact I had an issue in case systemd was used instead of sysV but I can't reproduce this anymore after a full sstate clean.. I also had the same issue as you mention when using sysV.

So I clearly didn't tested the patch enough, sorry about that.

Fabien

________________________________________
From: Richard Purdie [richard.purdie@linuxfoundation.org]
Sent: Sunday, June 29, 2014 11:05 AM
To: Chereau, Fabien
Cc: Openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] libusb: add missing build dependency on udev

On Thu, 2014-06-26 at 12:20 +0200, Fabien Chereau wrote:
> This fixes the ./configure error in case libusb is built before udev/systemd.
>
> Signed-off-by: Fabien Chereau <fabien.chereau@intel.com>
> ---
>  meta/recipes-support/libusb/libusb1_1.0.19.bb | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/meta/recipes-support/libusb/libusb1_1.0.19.bb b/meta/recipes-support/libusb/libusb1_1.0.19.bb
> index a20e1fc..be2be9d 100644
> --- a/meta/recipes-support/libusb/libusb1_1.0.19.bb
> +++ b/meta/recipes-support/libusb/libusb1_1.0.19.bb
> @@ -6,6 +6,8 @@ SECTION = "libs"
>  LICENSE = "LGPLv2.1+"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=fbc093901857fcd118f065f900982c24"
>
> +DEPENDS_append = " udev"
> +
>  BBCLASSEXTEND = "native nativesdk"
>
>  SRC_URI = "${SOURCEFORGE_MIRROR}/libusb/libusb-${PV}.tar.bz2 \

This causes the build to explode with circular dependency issues. How
did you test this?

https://autobuilder.yoctoproject.org/main/builders/nightly-ppc-lsb/builds/147/steps/BuildImages/logs/stdio

There is a second error about a missing nativesdk-udev dependency (to
build nativesdk-libusb1) too.

Cheers,

Richard

---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris, 
92196 Meudon Cedex, France
Registration Number:  302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.