Skip to content

Commit 3107f8a

Browse files
author
Timmy
committed
Merge pull request #97 from lzamparo/master
Fix to enable successful build on MacOSX
2 parents 03d7e7e + 7a9e3df commit 3107f8a

File tree

1 file changed

+13
-6
lines changed

1 file changed

+13
-6
lines changed

src/CMakeLists.txt

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,13 @@ option( BUILD_SHARED_LIBS "Build shared libraries" ON )
2727

2828
# By default test-correctness is linked and tested against ACML library.
2929
# However, test-correctness can instead use NETLIB as a reference library
30-
set(CORR_TEST_WITH_ACML ON CACHE BOOL "Use ACML library in correctness tests")
30+
# On Mac OSX systems, this must be set to OFF for the build to succeed (due to nesting of FindBLAS code)
31+
if ( APPLE )
32+
set(CORR_TEST_WITH_ACML OFF CACHE BOOL "Use ACML library in correctness tests")
33+
else ( )
34+
message(STATUS "CORR_TEST_WITH_ACML set to ON")
35+
set(CORR_TEST_WITH_ACML ON CACHE BOOL "Use ACML library in correctness tests")
36+
endif( )
3137

3238
if( CMAKE_GENERATOR MATCHES "NMake" )
3339
option( NMAKE_COMPILE_VERBOSE "Print compile and link strings to the console" OFF )
@@ -121,14 +127,15 @@ endif()
121127
# TODO: maybe this could be written using the FindBLAS module in the future
122128
if( BUILD_TEST )
123129
if(NOT CORR_TEST_WITH_ACML)
124-
if(APPLE)
125-
find_library(BLAS_LIBRARIES Accelerate)
130+
if(APPLE)
131+
message(STATUS "Trying to find Accelerate library")
132+
find_library(BLAS_LIBRARIES Accelerate HINTS /System/Library/Frameworks/Accelerate.framework)
126133
MARK_AS_ADVANCED(BLAS_LIBRARIES)
127134
message(STATUS "Using Accelerate framework on Mac OS-X")
128-
else()
135+
else()
129136
find_package( Netlib COMPONENTS BLAS REQUIRED )
130-
endif()
131-
else( )
137+
endif()
138+
else( )
132139
# Find ACML BLAS implementation
133140
# platform dependent ACML subdirectory
134141
if (WIN32)

0 commit comments

Comments
 (0)