Browse Source

Find{BLAS,LAPACK}: Avoid repeated setting of prefered library suffixes

Mario Emmenlauer 6 years ago
parent
commit
fde5fcbc71
2 changed files with 28 additions and 28 deletions
  1. 14 14
      Modules/FindBLAS.cmake
  2. 14 14
      Modules/FindLAPACK.cmake

+ 14 - 14
Modules/FindBLAS.cmake

@@ -117,8 +117,6 @@ include(${CMAKE_CURRENT_LIST_DIR}/FindPackageHandleStandardArgs.cmake)
 cmake_push_check_state()
 cmake_push_check_state()
 set(CMAKE_REQUIRED_QUIET ${BLAS_FIND_QUIETLY})
 set(CMAKE_REQUIRED_QUIET ${BLAS_FIND_QUIETLY})
 
 
-set(_blas_ORIG_CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES})
-
 if(BLA_PREFER_PKGCONFIG)
 if(BLA_PREFER_PKGCONFIG)
   find_package(PkgConfig)
   find_package(PkgConfig)
   pkg_check_modules(PKGC_BLAS blas)
   pkg_check_modules(PKGC_BLAS blas)
@@ -129,6 +127,20 @@ if(BLA_PREFER_PKGCONFIG)
   endif()
   endif()
 endif()
 endif()
 
 
+set(_blas_ORIG_CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES})
+if(BLA_STATIC)
+  if(WIN32)
+    set(CMAKE_FIND_LIBRARY_SUFFIXES .lib ${CMAKE_FIND_LIBRARY_SUFFIXES})
+  else()
+    set(CMAKE_FIND_LIBRARY_SUFFIXES .a ${CMAKE_FIND_LIBRARY_SUFFIXES})
+  endif()
+else()
+  if(CMAKE_SYSTEM_NAME STREQUAL "Linux")
+    # for ubuntu's libblas3gf and liblapack3gf packages
+    set(CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES} .so.3gf)
+  endif()
+endif()
+
 # TODO: move this stuff to a separate module
 # TODO: move this stuff to a separate module
 
 
 macro(CHECK_BLAS_LIBRARIES LIBRARIES _prefix _name _flags _list _threadlibs _addlibdir _subdirs)
 macro(CHECK_BLAS_LIBRARIES LIBRARIES _prefix _name _flags _list _threadlibs _addlibdir _subdirs)
@@ -170,18 +182,6 @@ macro(CHECK_BLAS_LIBRARIES LIBRARIES _prefix _name _flags _list _threadlibs _add
         set(_combined_name ${_combined_name}_threadlibs)
         set(_combined_name ${_combined_name}_threadlibs)
       endif()
       endif()
       if(_libraries_work)
       if(_libraries_work)
-        if(BLA_STATIC)
-          if(WIN32)
-            set(CMAKE_FIND_LIBRARY_SUFFIXES .lib ${CMAKE_FIND_LIBRARY_SUFFIXES})
-          else()
-            set(CMAKE_FIND_LIBRARY_SUFFIXES .a ${CMAKE_FIND_LIBRARY_SUFFIXES})
-          endif()
-        else()
-          if(CMAKE_SYSTEM_NAME STREQUAL "Linux")
-            # for ubuntu's libblas3gf and liblapack3gf packages
-            set(CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES} .so.3gf)
-          endif()
-        endif()
         find_library(${_prefix}_${_library}_LIBRARY
         find_library(${_prefix}_${_library}_LIBRARY
           NAMES ${_library}
           NAMES ${_library}
           PATHS ${_addlibdir}
           PATHS ${_addlibdir}

+ 14 - 14
Modules/FindLAPACK.cmake

@@ -94,11 +94,23 @@ include(${CMAKE_CURRENT_LIST_DIR}/CMakePushCheckState.cmake)
 cmake_push_check_state()
 cmake_push_check_state()
 set(CMAKE_REQUIRED_QUIET ${LAPACK_FIND_QUIETLY})
 set(CMAKE_REQUIRED_QUIET ${LAPACK_FIND_QUIETLY})
 
 
-set(_lapack_ORIG_CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES})
-
 set(LAPACK_FOUND FALSE)
 set(LAPACK_FOUND FALSE)
 set(LAPACK95_FOUND FALSE)
 set(LAPACK95_FOUND FALSE)
 
 
+set(_lapack_ORIG_CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES})
+if(BLA_STATIC)
+  if(WIN32)
+    set(CMAKE_FIND_LIBRARY_SUFFIXES .lib ${CMAKE_FIND_LIBRARY_SUFFIXES})
+  else()
+    set(CMAKE_FIND_LIBRARY_SUFFIXES .a ${CMAKE_FIND_LIBRARY_SUFFIXES})
+  endif()
+else()
+  if(CMAKE_SYSTEM_NAME STREQUAL "Linux")
+    # for ubuntu's libblas3gf and liblapack3gf packages
+    set(CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES} .so.3gf)
+  endif()
+endif()
+
 # TODO: move this stuff to a separate module
 # TODO: move this stuff to a separate module
 
 
 macro(CHECK_LAPACK_LIBRARIES LIBRARIES _prefix _name _flags _list _threadlibs _addlibdir _subdirs _blas)
 macro(CHECK_LAPACK_LIBRARIES LIBRARIES _prefix _name _flags _list _threadlibs _addlibdir _subdirs _blas)
@@ -137,18 +149,6 @@ macro(CHECK_LAPACK_LIBRARIES LIBRARIES _prefix _name _flags _list _threadlibs _a
     else()
     else()
       set(_combined_name ${_combined_name}_${_library})
       set(_combined_name ${_combined_name}_${_library})
       if(_libraries_work)
       if(_libraries_work)
-        if(BLA_STATIC)
-          if(WIN32)
-            set(CMAKE_FIND_LIBRARY_SUFFIXES .lib ${CMAKE_FIND_LIBRARY_SUFFIXES})
-          else()
-            set(CMAKE_FIND_LIBRARY_SUFFIXES .a ${CMAKE_FIND_LIBRARY_SUFFIXES})
-          endif()
-        else()
-          if(CMAKE_SYSTEM_NAME STREQUAL "Linux")
-            # for ubuntu's libblas3gf and liblapack3gf packages
-            set(CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES} .so.3gf)
-          endif()
-        endif()
         find_library(${_prefix}_${_library}_LIBRARY
         find_library(${_prefix}_${_library}_LIBRARY
           NAMES ${_library}
           NAMES ${_library}
           PATHS ${_addlibdir}
           PATHS ${_addlibdir}