[Buildroot] [PATCH] clapack: Disable for MIPS platforms

Thomas De Schampheleire patrickdepinguin at gmail.com
Tue May 6 11:52:06 UTC 2014


Hi Vicente,

On Tue, May 6, 2014 at 1:11 PM, Vicente Olivert Riera
<Vincent.Riera at imgtec.com> wrote:
> Disable this package for MIPS because it needs IRIX headers and
> libraries.
>
> Fixes:
>    http://autobuild.buildroot.net/results/370/370f964441722675820c644403dde7fbc683a315/
>
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
> Reviewed-by: Markos Chandras <Markos.Chandras at imgtec.com>
> ---
>  package/armadillo/Config.in |    1 +
>  package/clapack/Config.in   |    1 +
>  2 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/package/armadillo/Config.in b/package/armadillo/Config.in
> index 2251d91..0b86b77 100644
> --- a/package/armadillo/Config.in
> +++ b/package/armadillo/Config.in
> @@ -5,6 +5,7 @@ config BR2_PACKAGE_ARMADILLO
>         bool "armadillo"
>         depends on BR2_INSTALL_LIBSTDCPP
>         depends on BR2_LARGEFILE # clapack
> +       depends on !(BR2_mips || BR2_mipsel || BR2_mips64 || BR2_mips64el) # clapack
>         select BR2_PACKAGE_CLAPACK
>         help
>           Armadillo: An Open Source C++ Linear Algebra Library for
> diff --git a/package/clapack/Config.in b/package/clapack/Config.in
> index a77094f..95dd343 100644
> --- a/package/clapack/Config.in
> +++ b/package/clapack/Config.in
> @@ -1,6 +1,7 @@
>  config BR2_PACKAGE_CLAPACK
>         bool "cblas/clapack"
>         depends on BR2_LARGEFILE
> +       depends on !(BR2_mips || BR2_mipsel || BR2_mips64 || BR2_mips64el)
>         help
>           BLAS and LAPACK C implementation (f2c'ed version of).
>

These dependencies should also be added to the respective comment lines, see
http://buildroot.uclibc.org/downloads/manual/manual.html#dependencies-target-toolchain-options

"The comment should only be visible if the config option itself would
be visible when the toolchain option dependencies are met. This means
that all other dependencies of the package (including dependencies on
target architecture and MMU support) have to be repeated on the
comment definition. To keep it clear, the depends on statement for
these non-toolchain option should be kept separate from the depends on
statement for the toolchain options. If there is a dependency on a
config option in that same file (typically the main package) it is
preferable to have a global if … endif construct rather than repeating
the depends on statement on the comment and other config options."

Best regards,
Thomas


More information about the buildroot mailing list