diff options
| author | Brad King <brad.king@kitware.com> | 2023-04-03 13:58:58 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2023-04-03 13:58:58 (GMT) |
| commit | 9d18426c479b9f337314c5fd1f8a3eaeaba89019 (patch) | |
| tree | 70c976d7d4137206e1ef01a319726c5c669d32b4 | |
| parent | 2c59598b9117f546e35f2cdb291c3052f75dfd3e (diff) | |
| download | CMake-9d18426c479b9f337314c5fd1f8a3eaeaba89019.zip CMake-9d18426c479b9f337314c5fd1f8a3eaeaba89019.tar.gz CMake-9d18426c479b9f337314c5fd1f8a3eaeaba89019.tar.bz2 | |
Find{BLAS,LAPACK}: Prefer BLA_VENDOR cmake variable over env variable
This is implied by the documentation, but was not implemented properly.
Fixes: #24664
| -rw-r--r-- | Modules/FindBLAS.cmake | 8 | ||||
| -rw-r--r-- | Modules/FindLAPACK.cmake | 10 |
2 files changed, 10 insertions, 8 deletions
diff --git a/Modules/FindBLAS.cmake b/Modules/FindBLAS.cmake index 7af1017..19bef94 100644 --- a/Modules/FindBLAS.cmake +++ b/Modules/FindBLAS.cmake @@ -386,10 +386,10 @@ set(BLAS_LINKER_FLAGS) set(BLAS_LIBRARIES) set(BLAS95_LIBRARIES) set(_blas_fphsa_req_var BLAS_LIBRARIES) -if(NOT $ENV{BLA_VENDOR} STREQUAL "") - set(BLA_VENDOR $ENV{BLA_VENDOR}) -else() - if(NOT BLA_VENDOR) +if(NOT BLA_VENDOR) + if(NOT "$ENV{BLA_VENDOR}" STREQUAL "") + set(BLA_VENDOR "$ENV{BLA_VENDOR}") + else() set(BLA_VENDOR "All") endif() endif() diff --git a/Modules/FindLAPACK.cmake b/Modules/FindLAPACK.cmake index 1eecb1c..17117bd 100644 --- a/Modules/FindLAPACK.cmake +++ b/Modules/FindLAPACK.cmake @@ -304,10 +304,12 @@ endif() # Search for different LAPACK distributions if BLAS is found if(NOT LAPACK_NOT_FOUND_MESSAGE) set(LAPACK_LINKER_FLAGS ${BLAS_LINKER_FLAGS}) - if(NOT $ENV{BLA_VENDOR} STREQUAL "") - set(BLA_VENDOR $ENV{BLA_VENDOR}) - elseif(NOT BLA_VENDOR) - set(BLA_VENDOR "All") + if(NOT BLA_VENDOR) + if(NOT "$ENV{BLA_VENDOR}" STREQUAL "") + set(BLA_VENDOR "$ENV{BLA_VENDOR}") + else() + set(BLA_VENDOR "All") + endif() endif() # LAPACK in the Intel MKL 10+ library? |