All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 0/3] mplayer autobuild fixes
@ 2014-02-19  9:53 Fabio Porcedda
  2014-02-19  9:53 ` [Buildroot] [PATCH 1/3] mplayer: not available on bfin Fabio Porcedda
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Fabio Porcedda @ 2014-02-19  9:53 UTC (permalink / raw)
  To: buildroot

Some mplayer autobuild fixes.

Best regards

Fabio Porcedda (3):
  mplayer: not available on bfin
  mplayer: remove sh2 support
  mplayer: add missing reverse dependencies for bfin and sh2a

 package/libplayer/Config.in | 3 ++-
 package/mplayer/Config.in   | 6 +++---
 2 files changed, 5 insertions(+), 4 deletions(-)

-- 
1.8.5.2

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 1/3] mplayer: not available on bfin
  2014-02-19  9:53 [Buildroot] [PATCH 0/3] mplayer autobuild fixes Fabio Porcedda
@ 2014-02-19  9:53 ` Fabio Porcedda
  2014-02-19 10:09   ` Thomas Petazzoni
  2014-02-19 14:53   ` Peter Korsgaard
  2014-02-19  9:53 ` [Buildroot] [PATCH 2/3] mplayer: remove sh2 support Fabio Porcedda
  2014-02-19  9:53 ` [Buildroot] [PATCH 3/3] mplayer: add missing reverse dependencies for bfin and sh2a Fabio Porcedda
  2 siblings, 2 replies; 9+ messages in thread
From: Fabio Porcedda @ 2014-02-19  9:53 UTC (permalink / raw)
  To: buildroot

Fixes:
http://autobuild.buildroot.net/results/109/109b7c9b575c7362c4982a908a1e3695e7590ebd/

Signed-off-by: Fabio Porcedda <fabio.porcedda@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)
 	# Broken support for <ARMv5
 	depends on !(BR2_arm7tdmi || BR2_arm720t || BR2_arm920t \
@@ -29,6 +29,6 @@ config BR2_PACKAGE_MPLAYER_MENCODER
 endif
 
 comment "mplayer needs a toolchain w/ largefile"
-	depends on !(BR2_sh2 || BR2_sh2a || BR2_sh4a || BR2_sh4aeb \
+	depends on !(BR2_bfin || BR2_sh2 || BR2_sh2a || BR2_sh4a || BR2_sh4aeb \
 			|| BR2_microblaze || BR2_aarch64)
 	depends on !BR2_LARGEFILE
-- 
1.8.5.2

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 2/3] mplayer: remove sh2 support
  2014-02-19  9:53 [Buildroot] [PATCH 0/3] mplayer autobuild fixes Fabio Porcedda
  2014-02-19  9:53 ` [Buildroot] [PATCH 1/3] mplayer: not available on bfin Fabio Porcedda
@ 2014-02-19  9:53 ` Fabio Porcedda
  2014-02-19 14:54   ` Peter Korsgaard
  2014-02-19  9:53 ` [Buildroot] [PATCH 3/3] mplayer: add missing reverse dependencies for bfin and sh2a Fabio Porcedda
  2 siblings, 1 reply; 9+ messages in thread
From: Fabio Porcedda @ 2014-02-19  9:53 UTC (permalink / raw)
  To: buildroot

Remove a leftover of the commit commit e543f5a10488b020bb4ccb03e6de0e455c64841a
(arch: remove sh2, sh3 and sh3eb support).

Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
---
 package/mplayer/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/mplayer/Config.in b/package/mplayer/Config.in
index f2b3620..cfe824d 100644
--- a/package/mplayer/Config.in
+++ b/package/mplayer/Config.in
@@ -29,6 +29,6 @@ config BR2_PACKAGE_MPLAYER_MENCODER
 endif
 
 comment "mplayer needs a toolchain w/ largefile"
-	depends on !(BR2_bfin || BR2_sh2 || BR2_sh2a || BR2_sh4a || BR2_sh4aeb \
+	depends on !(BR2_bfin || BR2_sh2a || BR2_sh4a || BR2_sh4aeb \
 			|| BR2_microblaze || BR2_aarch64)
 	depends on !BR2_LARGEFILE
-- 
1.8.5.2

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 3/3] mplayer: add missing reverse dependencies for bfin and sh2a
  2014-02-19  9:53 [Buildroot] [PATCH 0/3] mplayer autobuild fixes Fabio Porcedda
  2014-02-19  9:53 ` [Buildroot] [PATCH 1/3] mplayer: not available on bfin Fabio Porcedda
  2014-02-19  9:53 ` [Buildroot] [PATCH 2/3] mplayer: remove sh2 support Fabio Porcedda
@ 2014-02-19  9:53 ` Fabio Porcedda
  2014-02-19 14:54   ` Peter Korsgaard
  2 siblings, 1 reply; 9+ messages in thread
From: Fabio Porcedda @ 2014-02-19  9:53 UTC (permalink / raw)
  To: buildroot

Propagate to comment and libplayer unsupported platforms bfin and sh2a.

Fixes:
http://autobuild.buildroot.net/results/5d8/5d80ade791319b6a66ac1c342f7b1d25c023f0af/

Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
---
 package/libplayer/Config.in | 4 ++--
 package/mplayer/Config.in   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/package/libplayer/Config.in b/package/libplayer/Config.in
index 84dad1d..c8271a1 100644
--- a/package/libplayer/Config.in
+++ b/package/libplayer/Config.in
@@ -13,8 +13,8 @@ if BR2_PACKAGE_LIBPLAYER
 config BR2_PACKAGE_LIBPLAYER_MPLAYER
 	bool "mplayer backend"
 	# mplayer
-	depends on !(BR2_bfin || BR2_sh4a || BR2_sh4aeb || BR2_microblaze || \
-		   BR2_aarch64)
+	depends on !(BR2_bfin || BR2_sh2a || BR2_sh4a || BR2_sh4aeb || \
+		   BR2_microblaze || BR2_aarch64 || BR2_nios)
 	# mplayer
 	depends on BR2_LARGEFILE
 	# mplayer
diff --git a/package/mplayer/Config.in b/package/mplayer/Config.in
index cfe824d..bacd744 100644
--- a/package/mplayer/Config.in
+++ b/package/mplayer/Config.in
@@ -30,5 +30,5 @@ endif
 
 comment "mplayer needs a toolchain w/ largefile"
 	depends on !(BR2_bfin || BR2_sh2a || BR2_sh4a || BR2_sh4aeb \
-			|| BR2_microblaze || BR2_aarch64)
+			|| BR2_microblaze || BR2_aarch64 || BR2_nios2)
 	depends on !BR2_LARGEFILE
-- 
1.8.5.2

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 1/3] mplayer: not available on bfin
  2014-02-19  9:53 ` [Buildroot] [PATCH 1/3] mplayer: not available on bfin Fabio Porcedda
@ 2014-02-19 10:09   ` Thomas Petazzoni
  2014-02-19 14:55     ` Peter Korsgaard
  2014-02-19 14:53   ` Peter Korsgaard
  1 sibling, 1 reply; 9+ messages in thread
From: Thomas Petazzoni @ 2014-02-19 10:09 UTC (permalink / raw)
  To: buildroot

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@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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 1/3] mplayer: not available on bfin
  2014-02-19  9:53 ` [Buildroot] [PATCH 1/3] mplayer: not available on bfin Fabio Porcedda
  2014-02-19 10:09   ` Thomas Petazzoni
@ 2014-02-19 14:53   ` Peter Korsgaard
  1 sibling, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2014-02-19 14:53 UTC (permalink / raw)
  To: buildroot

>>>>> "Fabio" == Fabio Porcedda <fabio.porcedda@gmail.com> writes:

 > Fixes:
 > http://autobuild.buildroot.net/results/109/109b7c9b575c7362c4982a908a1e3695e7590ebd/

 > Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 2/3] mplayer: remove sh2 support
  2014-02-19  9:53 ` [Buildroot] [PATCH 2/3] mplayer: remove sh2 support Fabio Porcedda
@ 2014-02-19 14:54   ` Peter Korsgaard
  0 siblings, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2014-02-19 14:54 UTC (permalink / raw)
  To: buildroot

>>>>> "Fabio" == Fabio Porcedda <fabio.porcedda@gmail.com> writes:

 > Remove a leftover of the commit commit e543f5a10488b020bb4ccb03e6de0e455c64841a
 > (arch: remove sh2, sh3 and sh3eb support).

 > Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 3/3] mplayer: add missing reverse dependencies for bfin and sh2a
  2014-02-19  9:53 ` [Buildroot] [PATCH 3/3] mplayer: add missing reverse dependencies for bfin and sh2a Fabio Porcedda
@ 2014-02-19 14:54   ` Peter Korsgaard
  0 siblings, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2014-02-19 14:54 UTC (permalink / raw)
  To: buildroot

>>>>> "Fabio" == Fabio Porcedda <fabio.porcedda@gmail.com> writes:

 > Propagate to comment and libplayer unsupported platforms bfin and sh2a.
 > Fixes:
 > http://autobuild.buildroot.net/results/5d8/5d80ade791319b6a66ac1c342f7b1d25c023f0af/

 > Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>

s/bfin/nios2/

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 1/3] mplayer: not available on bfin
  2014-02-19 10:09   ` Thomas Petazzoni
@ 2014-02-19 14:55     ` Peter Korsgaard
  0 siblings, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2014-02-19 14:55 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:


 > 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>

I agree this would be a nice cleanup, but I've applied this series for
now for 2014.02.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2014-02-19 14:55 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-19  9:53 [Buildroot] [PATCH 0/3] mplayer autobuild fixes Fabio Porcedda
2014-02-19  9:53 ` [Buildroot] [PATCH 1/3] mplayer: not available on bfin Fabio Porcedda
2014-02-19 10:09   ` Thomas Petazzoni
2014-02-19 14:55     ` Peter Korsgaard
2014-02-19 14:53   ` Peter Korsgaard
2014-02-19  9:53 ` [Buildroot] [PATCH 2/3] mplayer: remove sh2 support Fabio Porcedda
2014-02-19 14:54   ` Peter Korsgaard
2014-02-19  9:53 ` [Buildroot] [PATCH 3/3] mplayer: add missing reverse dependencies for bfin and sh2a Fabio Porcedda
2014-02-19 14:54   ` Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.