[cmake-developers] [CMake 0012477]: FindLAPACK

[Mantis Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
http://public.kitware.com/Bug/view.php?id=12477 
====================================================================== 
Reported By:                Mark Abraham
Assigned To:                
====================================================================== 
Project:                    CMake
Issue ID:                   12477
Category:                   Modules
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
====================================================================== 
Date Submitted:             2011-09-30 03:20 EDT
Last Modified:              2011-09-30 03:20 EDT
====================================================================== 
Summary:                    FindLAPACK
Description: 
The linker command line produced for static linking with LAPACK on Unix is
broken. The semicolon is not a valid separator.

Steps to Reproduce: 
Call find_package(LAPACK) with UNIX and BLA_STATIC set, then watch linking die
later.

Additional Information: 
Recommended patch:

--- a/cmake/FindLAPACK.cmake
+++ b/cmake/FindLAPACK.cmake
@@ -107,7 +107,7 @@ endforeach(_library ${_list})
 if(_libraries_work)
   # Test this combination of libraries.
   if(UNIX AND BLA_STATIC)
-    set(CMAKE_REQUIRED_LIBRARIES ${_flags} "-Wl,--start-group ${${LIBRARIES}}
${_blas};-Wl,--end-group" ${_threads})
+    set(CMAKE_REQUIRED_LIBRARIES ${_flags} "-Wl,--start-group ${${LIBRARIES}}
${_blas} -Wl,--end-group" ${_threads})
   else(UNIX AND BLA_STATIC)
     set(CMAKE_REQUIRED_LIBRARIES ${_flags} ${${LIBRARIES}} ${_blas}
${_threads})
   endif(UNIX AND BLA_STATIC)

====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-09-30 03:20 Mark Abraham   New Issue                                    
======================================================================