* [PATCH] bluez5: remove udev dependency @ 2019-05-22 21:55 David Frey 2019-05-22 22:00 ` Joshua Watt 2019-05-22 22:03 ` Adrian Bunk 0 siblings, 2 replies; 6+ messages in thread From: David Frey @ 2019-05-22 21:55 UTC (permalink / raw) To: openembedded-core udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow users to decide if they want udev support. Signed-off-by: David Frey <dpfrey@gmail.com> --- meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc b/meta/recipes-connectivity/bluez5/bluez5.inc index aaf2af975d..568de86d6d 100644 --- a/meta/recipes-connectivity/bluez5/bluez5.inc +++ b/meta/recipes-connectivity/bluez5/bluez5.inc @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+" LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \ file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \ file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e" -DEPENDS = "udev dbus-glib glib-2.0" +DEPENDS = "dbus-glib glib-2.0" PROVIDES += "bluez-hcidump" RPROVIDES_${PN} += "bluez-hcidump" @@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = "--enable-threads,--disable-threads" PACKAGECONFIG[deprecated] = "--enable-deprecated,--disable-deprecated" PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell" PACKAGECONFIG[btpclient] = "--enable-btpclient,--disable-btpclient, ell" +PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev" SRC_URI = "\ ${KERNELORG_MIRROR}/linux/bluetooth/bluez-${PV}.tar.xz \ -- 2.21.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] bluez5: remove udev dependency 2019-05-22 21:55 [PATCH] bluez5: remove udev dependency David Frey @ 2019-05-22 22:00 ` Joshua Watt 2019-05-22 22:03 ` Adrian Bunk 1 sibling, 0 replies; 6+ messages in thread From: Joshua Watt @ 2019-05-22 22:00 UTC (permalink / raw) To: David Frey, openembedded-core On 5/22/19 4:55 PM, David Frey wrote: > udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow > users to decide if they want udev support. > > Signed-off-by: David Frey <dpfrey@gmail.com> > --- > meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc b/meta/recipes-connectivity/bluez5/bluez5.inc > index aaf2af975d..568de86d6d 100644 > --- a/meta/recipes-connectivity/bluez5/bluez5.inc > +++ b/meta/recipes-connectivity/bluez5/bluez5.inc > @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+" > LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \ > file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \ > file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e" > -DEPENDS = "udev dbus-glib glib-2.0" > +DEPENDS = "dbus-glib glib-2.0" > PROVIDES += "bluez-hcidump" > RPROVIDES_${PN} += "bluez-hcidump" Should it be enabled by default? > > @@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = "--enable-threads,--disable-threads" > PACKAGECONFIG[deprecated] = "--enable-deprecated,--disable-deprecated" > PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell" > PACKAGECONFIG[btpclient] = "--enable-btpclient,--disable-btpclient, ell" > +PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev" > > SRC_URI = "\ > ${KERNELORG_MIRROR}/linux/bluetooth/bluez-${PV}.tar.xz \ ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bluez5: remove udev dependency 2019-05-22 21:55 [PATCH] bluez5: remove udev dependency David Frey 2019-05-22 22:00 ` Joshua Watt @ 2019-05-22 22:03 ` Adrian Bunk 2019-05-23 16:12 ` David Frey 1 sibling, 1 reply; 6+ messages in thread From: Adrian Bunk @ 2019-05-22 22:03 UTC (permalink / raw) To: David Frey; +Cc: openembedded-core On Wed, May 22, 2019 at 02:55:41PM -0700, David Frey wrote: > udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow > users to decide if they want udev support. > > Signed-off-by: David Frey <dpfrey@gmail.com> > --- > meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc b/meta/recipes-connectivity/bluez5/bluez5.inc > index aaf2af975d..568de86d6d 100644 > --- a/meta/recipes-connectivity/bluez5/bluez5.inc > +++ b/meta/recipes-connectivity/bluez5/bluez5.inc > @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+" > LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \ > file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \ > file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e" > -DEPENDS = "udev dbus-glib glib-2.0" > +DEPENDS = "dbus-glib glib-2.0" > PROVIDES += "bluez-hcidump" > RPROVIDES_${PN} += "bluez-hcidump" > > @@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = "--enable-threads,--disable-threads" > PACKAGECONFIG[deprecated] = "--enable-deprecated,--disable-deprecated" > PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell" > PACKAGECONFIG[btpclient] = "--enable-btpclient,--disable-btpclient, ell" > +PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev" >... Unless there is a good reason to do otherwise, this should be added to the default PACKAGECONFIG so that the set of features enabled by default stays the same. cu Adrian -- "Is there not promise of rain?" Ling Tan asked suddenly out of the darkness. There had been need of rain for many days. "Only a promise," Lao Er said. Pearl S. Buck - Dragon Seed ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bluez5: remove udev dependency 2019-05-22 22:03 ` Adrian Bunk @ 2019-05-23 16:12 ` David Frey 2019-05-23 16:30 ` Joshua Watt 0 siblings, 1 reply; 6+ messages in thread From: David Frey @ 2019-05-23 16:12 UTC (permalink / raw) To: Adrian Bunk; +Cc: openembedded-core On 5/22/2019 3:03 PM, Adrian Bunk wrote: > On Wed, May 22, 2019 at 02:55:41PM -0700, David Frey wrote: >> udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow >> users to decide if they want udev support. >> >> Signed-off-by: David Frey <dpfrey@gmail.com> >> --- >> meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc b/meta/recipes-connectivity/bluez5/bluez5.inc >> index aaf2af975d..568de86d6d 100644 >> --- a/meta/recipes-connectivity/bluez5/bluez5.inc >> +++ b/meta/recipes-connectivity/bluez5/bluez5.inc >> @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+" >> LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \ >> file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \ >> file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e" >> -DEPENDS = "udev dbus-glib glib-2.0" >> +DEPENDS = "dbus-glib glib-2.0" >> PROVIDES += "bluez-hcidump" >> RPROVIDES_${PN} += "bluez-hcidump" >> >> @@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = "--enable-threads,--disable-threads" >> PACKAGECONFIG[deprecated] = "--enable-deprecated,--disable-deprecated" >> PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell" >> PACKAGECONFIG[btpclient] = "--enable-btpclient,--disable-btpclient, ell" >> +PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev" >> ... > > Unless there is a good reason to do otherwise, this should be added to > the default PACKAGECONFIG so that the set of features enabled by default > stays the same. I think bluez5 should be built with udev support if udev is going to be part of the system, but I don't think udev should be brought in as a dependency of bluez5 by default. How can I express this? I *think* this is what is done for systemd in the PACKAGECONFIG with this fragment: ${@bb.utils.filter('DISTRO_FEATURES', 'systemd', d)} This uses DISTRO_FEATURES and based on this (https://www.yoctoproject.org/docs/2.7/mega-manual/mega-manual.html#ref-features-distro), it seems that there is no "udev" distro feature, so I'm not sure how to proceed. Thanks, David ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bluez5: remove udev dependency 2019-05-23 16:12 ` David Frey @ 2019-05-23 16:30 ` Joshua Watt 2019-05-23 16:45 ` David Frey 0 siblings, 1 reply; 6+ messages in thread From: Joshua Watt @ 2019-05-23 16:30 UTC (permalink / raw) To: David Frey, Adrian Bunk; +Cc: openembedded-core On 5/23/19 11:12 AM, David Frey wrote: > On 5/22/2019 3:03 PM, Adrian Bunk wrote: >> On Wed, May 22, 2019 at 02:55:41PM -0700, David Frey wrote: >>> udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow >>> users to decide if they want udev support. >>> >>> Signed-off-by: David Frey <dpfrey@gmail.com> >>> --- >>> meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++- >>> 1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc b/meta/recipes-connectivity/bluez5/bluez5.inc >>> index aaf2af975d..568de86d6d 100644 >>> --- a/meta/recipes-connectivity/bluez5/bluez5.inc >>> +++ b/meta/recipes-connectivity/bluez5/bluez5.inc >>> @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+" >>> LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \ >>> file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \ >>> file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e" >>> -DEPENDS = "udev dbus-glib glib-2.0" >>> +DEPENDS = "dbus-glib glib-2.0" >>> PROVIDES += "bluez-hcidump" >>> RPROVIDES_${PN} += "bluez-hcidump" >>> >>> @@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = "--enable-threads,--disable-threads" >>> PACKAGECONFIG[deprecated] = "--enable-deprecated,--disable-deprecated" >>> PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell" >>> PACKAGECONFIG[btpclient] = "--enable-btpclient,--disable-btpclient, ell" >>> +PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev" >>> ... >> Unless there is a good reason to do otherwise, this should be added to >> the default PACKAGECONFIG so that the set of features enabled by default >> stays the same. > I think bluez5 should be built with udev support if udev is going to be > part of the system, but I don't think udev should be brought in as a > dependency of bluez5 by default. How can I express this? I *think* > this is what is done for systemd in the PACKAGECONFIG with this fragment: > ${@bb.utils.filter('DISTRO_FEATURES', 'systemd', d)} I think it already is a dependency by default (before your patch). We would like this to remain the default for current consumers of bluez after your patch. I think the easiest way to do this is to have your patch add "udev" to the default PACKAGECONFIG for the recipe. Then, in your local.conf/distro.conf/whatever you can override the PACKAGECONFIG for the recipe to remove udev for your use case. This way, your patch allow you to have bluez without udev, but you won't break everyone else by suddenly removing it. > > This uses DISTRO_FEATURES and based on this > (https://www.yoctoproject.org/docs/2.7/mega-manual/mega-manual.html#ref-features-distro), > it seems that there is no "udev" distro feature, so I'm not sure how to > proceed. > > Thanks, > David ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bluez5: remove udev dependency 2019-05-23 16:30 ` Joshua Watt @ 2019-05-23 16:45 ` David Frey 0 siblings, 0 replies; 6+ messages in thread From: David Frey @ 2019-05-23 16:45 UTC (permalink / raw) To: Joshua Watt, Adrian Bunk; +Cc: openembedded-core On 5/23/2019 9:30 AM, Joshua Watt wrote: > > On 5/23/19 11:12 AM, David Frey wrote: >> On 5/22/2019 3:03 PM, Adrian Bunk wrote: >>> On Wed, May 22, 2019 at 02:55:41PM -0700, David Frey wrote: >>>> udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow >>>> users to decide if they want udev support. >>>> >>>> Signed-off-by: David Frey <dpfrey@gmail.com> >>>> --- >>>> meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++- >>>> 1 file changed, 2 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc >>>> b/meta/recipes-connectivity/bluez5/bluez5.inc >>>> index aaf2af975d..568de86d6d 100644 >>>> --- a/meta/recipes-connectivity/bluez5/bluez5.inc >>>> +++ b/meta/recipes-connectivity/bluez5/bluez5.inc >>>> @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+" >>>> LIC_FILES_CHKSUM = >>>> "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \ >>>> >>>> file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \ >>>> >>>> file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e" >>>> >>>> -DEPENDS = "udev dbus-glib glib-2.0" >>>> +DEPENDS = "dbus-glib glib-2.0" >>>> PROVIDES += "bluez-hcidump" >>>> RPROVIDES_${PN} += "bluez-hcidump" >>>> @@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = >>>> "--enable-threads,--disable-threads" >>>> PACKAGECONFIG[deprecated] = >>>> "--enable-deprecated,--disable-deprecated" >>>> PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell" >>>> PACKAGECONFIG[btpclient] = >>>> "--enable-btpclient,--disable-btpclient, ell" >>>> +PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev" >>>> ... >>> Unless there is a good reason to do otherwise, this should be added to >>> the default PACKAGECONFIG so that the set of features enabled by default >>> stays the same. >> I think bluez5 should be built with udev support if udev is going to be >> part of the system, but I don't think udev should be brought in as a >> dependency of bluez5 by default. How can I express this? I *think* >> this is what is done for systemd in the PACKAGECONFIG with this fragment: >> ${@bb.utils.filter('DISTRO_FEATURES', 'systemd', d)} > > I think it already is a dependency by default (before your patch). We > would like this to remain the default for current consumers of bluez > after your patch. I think the easiest way to do this is to have your > patch add "udev" to the default PACKAGECONFIG for the recipe. Then, in > your local.conf/distro.conf/whatever you can override the PACKAGECONFIG > for the recipe to remove udev for your use case. > > This way, your patch allow you to have bluez without udev, but you won't > break everyone else by suddenly removing it. I have submitted v2 based on your feedback. Thanks, David ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-05-23 16:45 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-05-22 21:55 [PATCH] bluez5: remove udev dependency David Frey 2019-05-22 22:00 ` Joshua Watt 2019-05-22 22:03 ` Adrian Bunk 2019-05-23 16:12 ` David Frey 2019-05-23 16:30 ` Joshua Watt 2019-05-23 16:45 ` David Frey
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox