[Buildroot] [PATCH 1/1] Added mpd-mpc, mpc client to mpd
thomas.petazzoni at free-electrons.com
Sat Dec 6 15:23:23 UTC 2014
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
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
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
More information about the buildroot