Explorar el Código

package/mpd: handle expat and yajl explicitly

mpd's Config.in selects expat and/or yajl depending on which
sub-options is enabled, and adds build dependencies, but does not pass
-D<option>=enabled/disabled for both of these features, relying on
auto-detection.

This commit fixes that by explicitly enabling/disabling expat and yajl
depending on their presence. The mpd sub-options no longer need to
have these as build-time dependencies, as it is handled globally.

Signed-off-by: Andreas Ziegler <br015@umbiko.net>
[Thomas: extracted from
https://patchwork.ozlabs.org/project/buildroot/patch/20221005091032.3014-5-br015@umbiko.net/]
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
(cherry picked from commit 3693462a1f8bd328c1a4f8fb68f00ce9127c31cd)
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
Andreas Ziegler hace 2 años
padre
commit
cd3b0fa07d
Se han modificado 1 ficheros con 15 adiciones y 4 borrados
  1. 15 4
      package/mpd/mpd.mk

+ 15 - 4
package/mpd/mpd.mk

@@ -29,6 +29,13 @@ else
 MPD_CONF_OPTS += -Dzeroconf=disabled
 MPD_CONF_OPTS += -Dzeroconf=disabled
 endif
 endif
 
 
+ifeq ($(BR2_PACKAGE_EXPAT),y)
+MPD_DEPENDENCIES += expat
+MPD_CONF_OPTS += -Dexpat=enabled
+else
+MPD_CONF_OPTS += -Dexpat=disabled
+endif
+
 # MPD prefers libicu for utf8 collation instead of libglib2.
 # MPD prefers libicu for utf8 collation instead of libglib2.
 ifeq ($(BR2_PACKAGE_ICU),y)
 ifeq ($(BR2_PACKAGE_ICU),y)
 MPD_DEPENDENCIES += icu
 MPD_DEPENDENCIES += icu
@@ -37,6 +44,13 @@ else
 MPD_CONF_OPTS += -Dicu=disabled
 MPD_CONF_OPTS += -Dicu=disabled
 endif
 endif
 
 
+ifeq ($(BR2_PACKAGE_YAJL),y)
+MPD_DEPENDENCIES += yajl
+MPD_CONF_OPTS += -Dyajl=enabled
+else
+MPD_CONF_OPTS += -Dyajl=disabled
+endif
+
 ifeq ($(BR2_PACKAGE_MPD_ALSA),y)
 ifeq ($(BR2_PACKAGE_MPD_ALSA),y)
 MPD_DEPENDENCIES += alsa-lib
 MPD_DEPENDENCIES += alsa-lib
 MPD_CONF_OPTS += -Dalsa=enabled
 MPD_CONF_OPTS += -Dalsa=enabled
@@ -251,7 +265,7 @@ MPD_CONF_OPTS += -Dpulse=disabled
 endif
 endif
 
 
 ifeq ($(BR2_PACKAGE_MPD_QOBUZ),y)
 ifeq ($(BR2_PACKAGE_MPD_QOBUZ),y)
-MPD_DEPENDENCIES += libgcrypt yajl
+MPD_DEPENDENCIES += libgcrypt
 MPD_CONF_OPTS += -Dqobuz=enabled
 MPD_CONF_OPTS += -Dqobuz=enabled
 else
 else
 MPD_CONF_OPTS += -Dqobuz=disabled
 MPD_CONF_OPTS += -Dqobuz=disabled
@@ -272,7 +286,6 @@ MPD_CONF_OPTS += -Dsidplay=disabled
 endif
 endif
 
 
 ifeq ($(BR2_PACKAGE_MPD_SOUNDCLOUD),y)
 ifeq ($(BR2_PACKAGE_MPD_SOUNDCLOUD),y)
-MPD_DEPENDENCIES += yajl
 MPD_CONF_OPTS += -Dsoundcloud=enabled
 MPD_CONF_OPTS += -Dsoundcloud=enabled
 else
 else
 MPD_CONF_OPTS += -Dsoundcloud=disabled
 MPD_CONF_OPTS += -Dsoundcloud=disabled
@@ -307,12 +320,10 @@ endif
 
 
 ifeq ($(BR2_PACKAGE_MPD_UPNP_PUPNP),y)
 ifeq ($(BR2_PACKAGE_MPD_UPNP_PUPNP),y)
 MPD_DEPENDENCIES += \
 MPD_DEPENDENCIES += \
-	expat \
 	libupnp
 	libupnp
 MPD_CONF_OPTS += -Dupnp=pupnp
 MPD_CONF_OPTS += -Dupnp=pupnp
 else ifeq ($(BR2_PACKAGE_MPD_UPNP_NPUPNP),y)
 else ifeq ($(BR2_PACKAGE_MPD_UPNP_NPUPNP),y)
 MPD_DEPENDENCIES += \
 MPD_DEPENDENCIES += \
-	expat \
 	libnpupnp
 	libnpupnp
 MPD_CONF_OPTS += -Dupnp=npupnp
 MPD_CONF_OPTS += -Dupnp=npupnp
 else ifeq ($(BR2_PACKAGE_MPD_UPNP_DISABLED),y)
 else ifeq ($(BR2_PACKAGE_MPD_UPNP_DISABLED),y)