[OE-core] [PATCH] bluez4: make alsa support conditional upon DISTRO_FEATURES
Saul Wold
sgw at linux.intel.com
Wed Apr 18 22:44:16 UTC 2012
On 04/18/2012 01:30 PM, Jonas Danielsson wrote:
> Do not enable alsa in bluez4 unless it's included in DISTRO_FEATURES.
>
> Signed-off-by: Jonas Danielsson<jonas.danielsson at lundinova.se>
> ---
> meta/recipes-connectivity/bluez/bluez4.inc | 7 +++++--
> meta/recipes-connectivity/bluez/bluez4_4.99.bb | 11 +++++++----
> 2 files changed, 12 insertions(+), 6 deletions(-)
>
> diff --git a/meta/recipes-connectivity/bluez/bluez4.inc b/meta/recipes-connectivity/bluez/bluez4.inc
> index fbbe0bf..0ce7c31 100644
> --- a/meta/recipes-connectivity/bluez/bluez4.inc
> +++ b/meta/recipes-connectivity/bluez/bluez4.inc
> @@ -7,7 +7,10 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
> file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \
> file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e \
> file://sbc/sbc.c;beginline=1;endline=25;md5=1a40781ed30d50d8639323a184aeb191"
> -DEPENDS = "udev alsa-lib libusb dbus-glib glib-2.0 libcheck"
> +DEPENDS = "\
> + udev ${@base_contains('DISTRO_FEATURES', 'alsa', 'alsa-lib', '', d)} \
> + libusb dbus-glib glib-2.0 libcheck \
> + "
> RDEPENDS_${PN}-dev = "bluez-hcidump"
>
> ASNEEDED = ""
> @@ -21,7 +24,7 @@ inherit autotools
>
> EXTRA_OECONF = "\
> --disable-gstreamer \
> - --enable-alsa \
> + ${@base_contains('DISTRO_FEATURES', 'alsa', '--enable-alsa', '--disable-alsa', d)} \
> --enable-usb \
> --enable-tools \
> --enable-bccmd \
Thanks for the contribution, there are some adjustments that I would
suggest making to more maintainable.
You should be using PACKAGECONFIG here instead of base_contains.
> diff --git a/meta/recipes-connectivity/bluez/bluez4_4.99.bb b/meta/recipes-connectivity/bluez/bluez4_4.99.bb
> index 5033d34..5f1ef45 100644
> --- a/meta/recipes-connectivity/bluez/bluez4_4.99.bb
> +++ b/meta/recipes-connectivity/bluez/bluez4_4.99.bb
> @@ -1,6 +1,6 @@
> require bluez4.inc
>
> -PR = "r1"
> +PR = "r2"
>
> SRC_URI += "file://bluetooth.conf \
> file://sbc_mmx.patch \
> @@ -19,13 +19,16 @@ do_install_append() {
>
> RDEPENDS_${PN}-dev = "bluez-hcidump"
>
> -PACKAGES =+ "libasound-module-bluez"
> +PACKAGES =+ "${@base_contains('DISTRO_FEATURES', 'alsa', 'libasound-module-bluez', '', d)}"
>
> -FILES_libasound-module-bluez = "${libdir}/alsa-lib/lib*.so ${datadir}/alsa"
> +FILES_libasound-module-bluez = "\
> + ${@base_contains('DISTRO_FEATURES', 'alsa', '${libdir}/alsa-lib/lib*.so', '', d)} \
> + ${@base_contains('DISTRO_FEATURES', 'alsa', '${datadir}/alsa', '', d)} \
> +"
Also, you can use ALLOW_EMPTY_libasound-module-bluez = "1" so that it
does not matter if the files are available
> FILES_${PN} += "${libdir}/bluetooth/plugins ${libdir}/bluetooth/plugins/*.so ${base_libdir}/udev/ ${systemd_unitdir}/"
> FILES_${PN}-dev += "\
> ${libdir}/bluetooth/plugins/*.la \
> - ${libdir}/alsa-lib/*.la \
> + ${@base_contains('DISTRO_FEATURES', 'alsa', '${libdir}/alsa-lib/*.la', '', d)} \
> "
>
The base_contains is not needed here, if the alsa-lib/*.la files are not
there then I don't think any warnings or errors will occur
> FILES_${PN}-dbg += "\
More information about the Openembedded-core
mailing list