[Buildroot] [PATCH 1/1] Added mpd-mpc, mpc client to mpd

[Buildroot] [PATCH 1/1] Added mpd-mpc, mpc client to mpd

[Thierry Bultel]

Signed-off-by: Thierry Bultel <tbultel@free.fr>
---
 package/Config.in          |  1 +
 package/mpd-mpc/Config.in  |  8 ++++++++
 package/mpd-mpc/mpd-mpc.mk | 12 ++++++++++++
 3 files changed, 21 insertions(+)
 create mode 100644 package/mpd-mpc/Config.in
 create mode 100644 package/mpd-mpc/mpd-mpc.mk

diff --git a/package/Config.in b/package/Config.in
index a8010d3..19eef96 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -19,6 +19,7 @@ menu "Audio and video applications"
 	source "package/madplay/Config.in"
 	source "package/modplugtools/Config.in"
 	source "package/mpd/Config.in"
+	source "package/mpd-mpc/Config.in"
 	source "package/mpg123/Config.in"
 	source "package/mplayer/Config.in"
 	source "package/musepack/Config.in"
diff --git a/package/mpd-mpc/Config.in b/package/mpd-mpc/Config.in
new file mode 100644
index 0000000..a56e2a6
--- /dev/null
+++ b/package/mpd-mpc/Config.in
@@ -0,0 +1,8 @@
+config BR2_PACKAGE_MPD_MPC
+	bool "mpd-mpc"
+	select BR2_PACKAGE_MPD
+	select BR2_PACKAGE_LIBMPDCLIENT
+	help
+	  A minimalist command line interface to MPD.
+
+	  http://www.musicpd.org/clients/mpc/
diff --git a/package/mpd-mpc/mpd-mpc.mk b/package/mpd-mpc/mpd-mpc.mk
new file mode 100644
index 0000000..3950a1e
--- /dev/null
+++ b/package/mpd-mpc/mpd-mpc.mk
@@ -0,0 +1,12 @@
+################################################################################
+#
+# mpc
+#
+################################################################################
+
+MPD_MPC_VERSION = 0.26
+MPD_MPC_SITE = http://www.musicpd.org/download/mpc/0
+MPD_MPC_SOURCE=mpc-$(MPD_MPC_VERSION).tar.xz
+MPD_MPC_DEPENDENCIES = mpd libmpdclient
+
+$(eval $(autotools-package))
-- 
1.9.1

[Buildroot] [PATCH 1/1] Added mpd-mpc, mpc client to mpd

[Thomas Petazzoni]

Dear Thierry Bultel,

On Sat,  6 Dec 2014 15:42:09 +0100, Thierry Bultel wrote:

> diff --git a/package/mpd-mpc/Config.in b/package/mpd-mpc/Config.in
> new file mode 100644
> index 0000000..a56e2a6
> --- /dev/null
> +++ b/package/mpd-mpc/Config.in
> @@ -0,0 +1,8 @@
> +config BR2_PACKAGE_MPD_MPC
> +	bool "mpd-mpc"
> +	select BR2_PACKAGE_MPD
> +	select BR2_PACKAGE_LIBMPDCLIENT

When you select a package, you must replicate the dependencies of this
package. So if you select BR2_PACKAGE_MPD, you should add:

        depends on BR2_INSTALL_LIBSTDCPP # mpd
        depends on BR2_LARGEFILE # mpd -> boost
        depends on BR2_USE_WCHAR # mpd -> libglib2, flac
        depends on BR2_TOOLCHAIN_HAS_THREADS # mpd -> libglib2
        depends on BR2_USE_MMU # mpd-> libglib2
        # avr32, sparc & CS powerpc gcc are too old for mpd
        depends on !(BR2_avr32 || BR2_sparc)
        depends on !(BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC201103 || BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC201009)

However, are you sure "mpd" is really a build dependency? To me, it
seems like only mpdclient is needed to build mpc. And you could imagine
having to build only the client on a given system, where the mpd server
would run on another system, no?

> +MPD_MPC_VERSION = 0.26
> +MPD_MPC_SITE = http://www.musicpd.org/download/mpc/0
> +MPD_MPC_SOURCE=mpc-$(MPD_MPC_VERSION).tar.xz

Spaces before and after = sign.

> +MPD_MPC_DEPENDENCIES = mpd libmpdclient

mpd dependency really needed?

You should also include MPD_MPC_LICENSE and MPD_MPC_LICENSE_FILES.

These days, we also like to have the hash file for new packages. See
http://buildroot.org/downloads/manual/manual.html#adding-packages-hash
for details.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com