[Buildroot] [PATCH 1/3] mplayer: not available on bfin
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Wed Feb 19 10:09:55 UTC 2014
Dear Fabio Porcedda,
On Wed, 19 Feb 2014 10:53:20 +0100, Fabio Porcedda wrote:
> Fixes:
> http://autobuild.buildroot.net/results/109/109b7c9b575c7362c4982a908a1e3695e7590ebd/
>
> Signed-off-by: Fabio Porcedda <fabio.porcedda at gmail.com>
> ---
> package/libplayer/Config.in | 3 ++-
> package/mplayer/Config.in | 4 ++--
> 2 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/package/libplayer/Config.in b/package/libplayer/Config.in
> index 113f650..84dad1d 100644
> --- a/package/libplayer/Config.in
> +++ b/package/libplayer/Config.in
> @@ -13,7 +13,8 @@ if BR2_PACKAGE_LIBPLAYER
> config BR2_PACKAGE_LIBPLAYER_MPLAYER
> bool "mplayer backend"
> # mplayer
> - depends on !BR2_sh4a && !BR2_sh4aeb && !BR2_microblaze && !BR2_aarch64
> + depends on !(BR2_bfin || BR2_sh4a || BR2_sh4aeb || BR2_microblaze || \
> + BR2_aarch64)
> # mplayer
> depends on BR2_LARGEFILE
> # mplayer
> diff --git a/package/mplayer/Config.in b/package/mplayer/Config.in
> index ae273e4..f2b3620 100644
> --- a/package/mplayer/Config.in
> +++ b/package/mplayer/Config.in
> @@ -1,7 +1,7 @@
> config BR2_PACKAGE_MPLAYER
> bool "mplayer"
> # Those architectures are not supported by MPlayer
> - depends on !(BR2_sh2a || BR2_sh4a || BR2_sh4aeb \
> + depends on !(BR2_bfin || BR2_sh2a || BR2_sh4a || BR2_sh4aeb \
> || BR2_microblaze || BR2_aarch64 || BR2_nios2)
I believe it's time to switch to a different logic. What about:
config BR2_PACKAGE_MPLAYER_ARCH_SUPPORTS
bool
depends on <list of supported architectures>
and then use "depends on BR2_PACKAGE_MPLAYER_ARCH_SUPPORTS" / "depends
on !BR2_PACKAGE_MPLAYER_ARCH_SUPPORTS" all over the place instead of
duplicating the list of architectures everywhere.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list