diff mbox

[2/2] package/mpd: drop dependency on libglib2

Message ID 20170115203435.908-2-joerg.krause@embedded.rocks
State Accepted
Commit d0774df0918d9a159ecab21cc2290f81bc4850f1
Headers show

Commit Message

Jörg Krause Jan. 15, 2017, 8:34 p.m. UTC
Version 0.20.x does not depend on libglib2 anymore. Note, that the
dependency on MMU and Thread remain, as MPD uses fork() and thread
functionality.

Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
---
 package/mpd/Config.in | 7 +++----
 package/mpd/mpd.mk    | 2 +-
 2 files changed, 4 insertions(+), 5 deletions(-)

Comments

Yann E. MORIN Jan. 15, 2017, 9:12 p.m. UTC | #1
Jörg, All,

On 2017-01-15 21:34 +0100, Jörg Krause spake thusly:
> Version 0.20.x does not depend on libglib2 anymore. Note, that the
> dependency on MMU and Thread remain, as MPD uses fork() and thread
> functionality.

This patch should be folded in the previous one.

Regards,
Yann E. MORIN.

> Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
> ---
>  package/mpd/Config.in | 7 +++----
>  package/mpd/mpd.mk    | 2 +-
>  2 files changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/package/mpd/Config.in b/package/mpd/Config.in
> index e0bb60447..9b22f68ce 100644
> --- a/package/mpd/Config.in
> +++ b/package/mpd/Config.in
> @@ -1,13 +1,12 @@
>  menuconfig BR2_PACKAGE_MPD
>  	bool "mpd"
>  	depends on BR2_INSTALL_LIBSTDCPP
> -	depends on BR2_USE_WCHAR # libglib2, flac
> -	depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
> -	depends on BR2_USE_MMU # libglib2
> +	depends on BR2_USE_WCHAR # flac
> +	depends on BR2_TOOLCHAIN_HAS_THREADS
> +	depends on BR2_USE_MMU # fork
>  	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C++14
>  	depends on BR2_TOOLCHAIN_HAS_ATOMIC
>  	select BR2_PACKAGE_BOOST
> -	select BR2_PACKAGE_LIBGLIB2
>  	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
>  	select BR2_PACKAGE_MPD_TREMOR if !(BR2_PACKAGE_MPD_MAD || BR2_PACKAGE_MPD_MPG123 || BR2_PACKAGE_MPD_VORBIS || BR2_PACKAGE_MPD_WAVPACK || BR2_PACKAGE_MPD_FLAC || BR2_PACKAGE_MPD_MUSEPACK || BR2_PACKAGE_MPD_FFMPEG)
>  	help
> diff --git a/package/mpd/mpd.mk b/package/mpd/mpd.mk
> index 82aca9665..446973730 100644
> --- a/package/mpd/mpd.mk
> +++ b/package/mpd/mpd.mk
> @@ -8,7 +8,7 @@ MPD_VERSION_MAJOR = 0.20
>  MPD_VERSION = $(MPD_VERSION_MAJOR).2
>  MPD_SOURCE = mpd-$(MPD_VERSION).tar.xz
>  MPD_SITE = http://www.musicpd.org/download/mpd/$(MPD_VERSION_MAJOR)
> -MPD_DEPENDENCIES = host-pkgconf boost libglib2
> +MPD_DEPENDENCIES = host-pkgconf boost
>  MPD_LICENSE = GPLv2+
>  MPD_LICENSE_FILES = COPYING
>  MPD_AUTORECONF = YES
> -- 
> 2.11.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Jörg Krause Jan. 16, 2017, 7:36 a.m. UTC | #2
Hi Yann,

On Sun, 2017-01-15 at 22:12 +0100, Yann E. MORIN wrote:
> Jörg, All,
> 
> On 2017-01-15 21:34 +0100, Jörg Krause spake thusly:
> > Version 0.20.x does not depend on libglib2 anymore. Note, that the
> > dependency on MMU and Thread remain, as MPD uses fork() and thread
> > functionality.
> 
> This patch should be folded in the previous one.

Note, that MPD has been bumped to version 0.20 some days ago. I agree,
that this would have been the right place to remove the dependency,
too. Now, it is just a bump from version 0.20 to 0.20.2. I seperated
the two issues to make more clear that libglib2 was *not* dropped in
0.20.2, but already in 0.20.

Jörg

> 
> > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
> > ---
> >  package/mpd/Config.in | 7 +++----
> >  package/mpd/mpd.mk    | 2 +-
> >  2 files changed, 4 insertions(+), 5 deletions(-)
> > 
> > diff --git a/package/mpd/Config.in b/package/mpd/Config.in
> > index e0bb60447..9b22f68ce 100644
> > --- a/package/mpd/Config.in
> > +++ b/package/mpd/Config.in
> > @@ -1,13 +1,12 @@
> >  menuconfig BR2_PACKAGE_MPD
> >  	bool "mpd"
> >  	depends on BR2_INSTALL_LIBSTDCPP
> > -	depends on BR2_USE_WCHAR # libglib2, flac
> > -	depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
> > -	depends on BR2_USE_MMU # libglib2
> > +	depends on BR2_USE_WCHAR # flac
> > +	depends on BR2_TOOLCHAIN_HAS_THREADS
> > +	depends on BR2_USE_MMU # fork
> >  	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C++14
> >  	depends on BR2_TOOLCHAIN_HAS_ATOMIC
> >  	select BR2_PACKAGE_BOOST
> > -	select BR2_PACKAGE_LIBGLIB2
> >  	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
> >  	select BR2_PACKAGE_MPD_TREMOR if !(BR2_PACKAGE_MPD_MAD ||
> > BR2_PACKAGE_MPD_MPG123 || BR2_PACKAGE_MPD_VORBIS ||
> > BR2_PACKAGE_MPD_WAVPACK || BR2_PACKAGE_MPD_FLAC ||
> > BR2_PACKAGE_MPD_MUSEPACK || BR2_PACKAGE_MPD_FFMPEG)
> >  	help
> > diff --git a/package/mpd/mpd.mk b/package/mpd/mpd.mk
> > index 82aca9665..446973730 100644
> > --- a/package/mpd/mpd.mk
> > +++ b/package/mpd/mpd.mk
> > @@ -8,7 +8,7 @@ MPD_VERSION_MAJOR = 0.20
> >  MPD_VERSION = $(MPD_VERSION_MAJOR).2
> >  MPD_SOURCE = mpd-$(MPD_VERSION).tar.xz
> >  MPD_SITE = http://www.musicpd.org/download/mpd/$(MPD_VERSION_MAJOR
> > )
> > -MPD_DEPENDENCIES = host-pkgconf boost libglib2
> > +MPD_DEPENDENCIES = host-pkgconf boost
> >  MPD_LICENSE = GPLv2+
> >  MPD_LICENSE_FILES = COPYING
> >  MPD_AUTORECONF = YES
> > -- 
> > 2.11.0
> > 
> > _______________________________________________
> > buildroot mailing list
> > buildroot@busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
> 
>
Peter Korsgaard Jan. 16, 2017, 12:57 p.m. UTC | #3
>>>>> "Jörg" == Jörg Krause <joerg.krause@embedded.rocks> writes:

 > Version 0.20.x does not depend on libglib2 anymore. Note, that the
 > dependency on MMU and Thread remain, as MPD uses fork() and thread
 > functionality.

 > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>

Committed, thanks.
diff mbox

Patch

diff --git a/package/mpd/Config.in b/package/mpd/Config.in
index e0bb60447..9b22f68ce 100644
--- a/package/mpd/Config.in
+++ b/package/mpd/Config.in
@@ -1,13 +1,12 @@ 
 menuconfig BR2_PACKAGE_MPD
 	bool "mpd"
 	depends on BR2_INSTALL_LIBSTDCPP
-	depends on BR2_USE_WCHAR # libglib2, flac
-	depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
-	depends on BR2_USE_MMU # libglib2
+	depends on BR2_USE_WCHAR # flac
+	depends on BR2_TOOLCHAIN_HAS_THREADS
+	depends on BR2_USE_MMU # fork
 	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C++14
 	depends on BR2_TOOLCHAIN_HAS_ATOMIC
 	select BR2_PACKAGE_BOOST
-	select BR2_PACKAGE_LIBGLIB2
 	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
 	select BR2_PACKAGE_MPD_TREMOR if !(BR2_PACKAGE_MPD_MAD || BR2_PACKAGE_MPD_MPG123 || BR2_PACKAGE_MPD_VORBIS || BR2_PACKAGE_MPD_WAVPACK || BR2_PACKAGE_MPD_FLAC || BR2_PACKAGE_MPD_MUSEPACK || BR2_PACKAGE_MPD_FFMPEG)
 	help
diff --git a/package/mpd/mpd.mk b/package/mpd/mpd.mk
index 82aca9665..446973730 100644
--- a/package/mpd/mpd.mk
+++ b/package/mpd/mpd.mk
@@ -8,7 +8,7 @@  MPD_VERSION_MAJOR = 0.20
 MPD_VERSION = $(MPD_VERSION_MAJOR).2
 MPD_SOURCE = mpd-$(MPD_VERSION).tar.xz
 MPD_SITE = http://www.musicpd.org/download/mpd/$(MPD_VERSION_MAJOR)
-MPD_DEPENDENCIES = host-pkgconf boost libglib2
+MPD_DEPENDENCIES = host-pkgconf boost
 MPD_LICENSE = GPLv2+
 MPD_LICENSE_FILES = COPYING
 MPD_AUTORECONF = YES