diff mbox

[1/2] package/cmake: disable on unsupported architectures

Message ID 1426583299-21207-1-git-send-email-s.martin49@gmail.com
State Superseded
Headers show

Commit Message

Samuel Martin March 17, 2015, 9:08 a.m. UTC
Supported architectures are listed at [1].

Fixes:
  http://autobuild.buildroot.net/results/de9/de9021fdc63600536ffa91e4b8718584e3e5702c/
  http://autobuild.buildroot.net/results/0f1/0f1afc62e91ab798fa65685af3dea3caac663f11/

[1] https://github.com/Kitware/CMake/blob/master/Utilities/KWIML/ABI.h.in#L339

Signed-off-by: Samuel Martin <s.martin49@gmail.com>
---
 package/cmake/Config.in | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Thomas Petazzoni March 17, 2015, 9:13 a.m. UTC | #1
Dear Samuel Martin,

On Tue, 17 Mar 2015 10:08:18 +0100, Samuel Martin wrote:

> diff --git a/package/cmake/Config.in b/package/cmake/Config.in
> index 0b5bba9..eee2461 100644
> --- a/package/cmake/Config.in
> +++ b/package/cmake/Config.in
> @@ -10,6 +10,12 @@ config BR2_PACKAGE_CMAKE_CTEST
>  	select BR2_PACKAGE_EXPAT
>  	select BR2_PACKAGE_BZIP2
>  	select BR2_PACKAGE_XZ
> +	depends on !BR2_arcle && !BR2_arceb
> +	depends on !BR2_bfin
> +	depends on !BR2_microblazeel && !BR2_microblazebe
> +	depends on !BR2_nios2
> +	depends on !BR2_sh && !BR2_sh64
> +	depends on !BR2_xtensa

So maybe we should have a positive test instead?

	depends on BR2_arm || BR2_armeb || BR2_...

Thomas
Baruch Siach March 17, 2015, 9:17 a.m. UTC | #2
Hi Samuel,

On Tue, Mar 17, 2015 at 10:08:18AM +0100, Samuel Martin wrote:
> Supported architectures are listed at [1].
> 
> Fixes:
>   http://autobuild.buildroot.net/results/de9/de9021fdc63600536ffa91e4b8718584e3e5702c/
>   http://autobuild.buildroot.net/results/0f1/0f1afc62e91ab798fa65685af3dea3caac663f11/
> 
> [1] https://github.com/Kitware/CMake/blob/master/Utilities/KWIML/ABI.h.in#L339
> 
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>
> ---
>  package/cmake/Config.in | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/package/cmake/Config.in b/package/cmake/Config.in
> index 0b5bba9..eee2461 100644
> --- a/package/cmake/Config.in
> +++ b/package/cmake/Config.in
> @@ -10,6 +10,12 @@ config BR2_PACKAGE_CMAKE_CTEST
>  	select BR2_PACKAGE_EXPAT
>  	select BR2_PACKAGE_BZIP2
>  	select BR2_PACKAGE_XZ
> +	depends on !BR2_arcle && !BR2_arceb
> +	depends on !BR2_bfin
> +	depends on !BR2_microblazeel && !BR2_microblazebe
> +	depends on !BR2_nios2
> +	depends on !BR2_sh && !BR2_sh64
> +	depends on !BR2_xtensa

Please update the dependencies comment below.

baruch
diff mbox

Patch

diff --git a/package/cmake/Config.in b/package/cmake/Config.in
index 0b5bba9..eee2461 100644
--- a/package/cmake/Config.in
+++ b/package/cmake/Config.in
@@ -10,6 +10,12 @@  config BR2_PACKAGE_CMAKE_CTEST
 	select BR2_PACKAGE_EXPAT
 	select BR2_PACKAGE_BZIP2
 	select BR2_PACKAGE_XZ
+	depends on !BR2_arcle && !BR2_arceb
+	depends on !BR2_bfin
+	depends on !BR2_microblazeel && !BR2_microblazebe
+	depends on !BR2_nios2
+	depends on !BR2_sh && !BR2_sh64
+	depends on !BR2_xtensa
 	depends on BR2_USE_WCHAR # libarchive
 	depends on BR2_INSTALL_LIBSTDCPP
 	help