mpd: remove version 0.19
authorJörg Krause <joerg.krause@embedded.rocks>
Wed, 13 Sep 2017 20:54:36 +0000 (22:54 +0200)
committerPeter Korsgaard <peter@korsgaard.com>
Fri, 22 Sep 2017 21:34:31 +0000 (23:34 +0200)
MPD version 0.20 is affected by GCC bug 64735 and failes to build for
some archictuctures (NIOSII, ARMv4, ARMv5 and SPARCv8) with a GCC
toolchain less then version 7.

We added a version choice between 0.19 and 0.20 in Buildroot version
2017.02 as GCC 7 was not yet available at that time. This way, mpd could
still be build for those architectures.

As GCC 7 is now available in Buildroot remove the version choice for 0.19.

Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
package/mpd/Config.in
package/mpd/mpd.hash
package/mpd/mpd.mk

index 0c73548fbbc0a47871dedb07eb99daa9cafe5c21..ffeef9b02b8462572f0747a488c5d98f37c81163 100644 (file)
@@ -5,9 +5,8 @@ menuconfig BR2_PACKAGE_MPD
        depends on BR2_TOOLCHAIN_HAS_THREADS
        depends on BR2_USE_MMU # fork
        depends on BR2_TOOLCHAIN_HAS_ATOMIC
-       # 0.19.x could be built with GCC 4.6, but for simplicity we require
-       # the same toolchain dependencies for all versions.
        depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C++14
+       depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # exception_ptr
        # ARC toolchain issue
        depends on !BR2_TOOLCHAIN_EXTERNAL_SYNOPSYS_ARC
        select BR2_PACKAGE_BOOST
@@ -23,24 +22,6 @@ menuconfig BR2_PACKAGE_MPD
 
 if BR2_PACKAGE_MPD
 
-# Hidden config symbols to select the branch 0.20 by default and fallback
-# to branch 0.19 for toolchains affected by GCC bug PR 64735. This way we
-# keep support for mpd on some architectures, like ARMv5, until GCC 7 is
-# released, which will fix this issue.
-config BR2_PACKAGE_MPD_VERSION_STRING
-       string
-       default "0.20" if !BR2_TOOLCHAIN_HAS_GCC_BUG_64735
-       default "0.19"
-
-config BR2_PACKAGE_MPD_VERSION_0_20
-       bool
-       default y if BR2_PACKAGE_MPD_VERSION_STRING = "0.20"
-
-config BR2_PACKAGE_MPD_VERSION_0_19
-       bool
-       default y if BR2_PACKAGE_MPD_VERSION_STRING = "0.19"
-       select BR2_PACKAGE_LIBGLIB2
-
 comment "Archive plugins"
 
 config BR2_PACKAGE_MPD_BZIP2
@@ -342,3 +323,6 @@ comment "mpd needs a toolchain w/ C++, threads, wchar, gcc >= 4.9"
        depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
                !BR2_TOOLCHAIN_HAS_THREADS || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
        depends on !BR2_TOOLCHAIN_EXTERNAL_SYNOPSYS_ARC
+
+comment "mpd needs a toolchain not affected by GCC bug 64735"
+       depends on BR2_TOOLCHAIN_HAS_GCC_BUG_64735
index 873286cda96aef6a37e5352748574e871484e5cc..9898c38c6eb984465ff17ff88722c5c2ec667b0e 100644 (file)
@@ -1,3 +1,2 @@
 # Locally calculated after checking pgp signature
 sha256  52fbc1125cdba41ba999add2820d45f3ce7cf493006bb04d8f0b2937204d3121  mpd-0.20.10.tar.xz
-sha256  8305b8bc026f4b6bde28b8dd09bfdddbe5590acf36358eed4d083a396e301730  mpd-0.19.21.tar.xz
index 692d402d3510a796c756f6f0d09ee7c84b322c1a..ff225939819789e75ae55006e18e33ee704b7fa8 100644 (file)
@@ -4,12 +4,8 @@
 #
 ################################################################################
 
-MPD_VERSION_MAJOR = $(call qstrip,$(BR2_PACKAGE_MPD_VERSION_STRING))
-ifeq ($(BR2_PACKAGE_MPD_VERSION_0_20),y)
+MPD_VERSION_MAJOR = 0.20
 MPD_VERSION = $(MPD_VERSION_MAJOR).10
-else
-MPD_VERSION = $(MPD_VERSION_MAJOR).21
-endif
 MPD_SOURCE = mpd-$(MPD_VERSION).tar.xz
 MPD_SITE = http://www.musicpd.org/download/mpd/$(MPD_VERSION_MAJOR)
 MPD_DEPENDENCIES = host-pkgconf boost
@@ -17,10 +13,6 @@ MPD_LICENSE = GPL-2.0+
 MPD_LICENSE_FILES = COPYING
 MPD_AUTORECONF = YES
 
-ifeq ($(BR2_PACKAGE_MPD_VERSION_0_19),y)
-MPD_DEPENDENCIES += libglib2
-endif
-
 # Some options need an explicit --disable or --enable
 
 # Zeroconf support depends on libdns_sd from avahi.