Skip to contentSkip to conversation

clapack: Disable for MIPS platforms

PATCH New Vicente Olivert Riera 2 participants 4 messages May 6, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 clapack: Disable for MIPS platforms

Disable this package for MIPS because it needs IRIX headers and libraries.

Fixes:
http://autobuild.buildroot.net/results/370/370f964441722675820c644403dde7fbc683a315/

---
 package/armadillo/Config.in |    1 +
 package/clapack/Config.in   |    1 +
 2 files changed, 2 insertions(+), 0 deletions(-)
package/armadillo/Config.in +1 -0
index 2251d91..0b86b77 100644
@@ -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
package/clapack/Config.in +1 -0
index a77094f..95dd343 100644
@@ -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).
 
Signature · 1 line
1.7.1
#1 lore raw

PATCH 1/1 clapack: Disable for MIPS platforms

Hi Vicente,

Quoted 30 lines · Vicente Olivert Riera

Disable this package for MIPS because it needs IRIX headers and libraries.

Fixes:

   http://autobuild.buildroot.net/results/370/370f964441722675820c644403dde7fbc683a315/
---
 package/armadillo/Config.in |    1 +
 package/clapack/Config.in   |    1 +
 2 files changed, 2 insertions(+), 0 deletions(-)
package/armadillo/Config.in +1 -0
index 2251d91..0b86b77 100644
@@ -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
package/clapack/Config.in +0 -0
index a77094f..95dd343 100644

package/clapack/Config.in line 7

Quoted 7 lines
package/clapack/Config.in +1 -0
@@ -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

#2 lore raw

PATCH 1/1 clapack: Disable for MIPS platforms

Quoted 56 lines · Thomas De Schampheleire

Hi Vicente,

Quoted 30 lines · Vicente Olivert Riera

Disable this package for MIPS because it needs IRIX headers and libraries.

Fixes:

    http://autobuild.buildroot.net/results/370/370f964441722675820c644403dde7fbc683a315/
---
  package/armadillo/Config.in |    1 +
  package/clapack/Config.in   |    1 +
  2 files changed, 2 insertions(+), 0 deletions(-)
package/armadillo/Config.in +1 -0
index 2251d91..0b86b77 100644
@@ -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
package/clapack/Config.in +0 -0
index a77094f..95dd343 100644

package/clapack/Config.in line 7

Quoted 7 lines
package/clapack/Config.in +1 -0
@@ -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."

Quoted 3 lines

Best regards,
Thomas

Hi Thomas,

it wasn't me who put the "depends on BR2_LARGEFILE" in that package. Anyway, I will add that fix to my patch.

Cheers,

Signature · 1 line
Vincent
#3 lore raw

PATCH 1/1 clapack: Disable for MIPS platforms

Quoted 67 lines · Vicente Olivert Riera
Quoted 62 lines · Thomas De Schampheleire

Hi Vicente,

Quoted 34 lines · Vicente Olivert Riera

Disable this package for MIPS because it needs IRIX headers and libraries.

Fixes:

http://autobuild.buildroot.net/results/370/370f964441722675820c644403dde7fbc683a315/

---
  package/armadillo/Config.in |    1 +
  package/clapack/Config.in   |    1 +
  2 files changed, 2 insertions(+), 0 deletions(-)
package/armadillo/Config.in +1 -0
index 2251d91..0b86b77 100644
@@ -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

package/clapack/Config.in +0 -0
index a77094f..95dd343 100644

package/clapack/Config.in line 7

Quoted 7 lines
package/clapack/Config.in +1 -0
@@ -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."

Quoted 3 lines

Best regards,
Thomas

Quoted 7 lines

Hi Thomas,

it wasn't me who put the "depends on BR2_LARGEFILE" in that package. Anyway, I will add that fix to my patch.

Cheers,

Hi again Thomas,

sorry about my last email. I thought you were talking about BR2_LARGEFILE. I have sent a V2 patch.

Signature · 1 line
Vincent