Skip to content

Commit 9c45ef3

Browse files
authored
BLD: Improve finding Armadillo by the installed library (#132)
* improve finding armadillo * fix condition * Fix if/else/endif style * remove checks, fix interface target * revert some inconsistent if style
1 parent 3bb34be commit 9c45ef3

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

cmake/carmaDependencyTargets.cmake

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,24 @@ IF (NOT DEFINED PYTHON_EXECUTABLE)
2525
ENDIF ()
2626

2727
# Armadillo dependency
28-
IF (NOT TARGET armadillo)
28+
IF (NOT TARGET armadillo AND NOT TARGET armadillo::armadillo)
2929
IF (carma_FIND_REQUIRED)
3030
FIND_PACKAGE(Armadillo REQUIRED)
3131
ELSE ()
3232
FIND_PACKAGE(Armadillo)
3333
ENDIF ()
34-
ADD_LIBRARY(armadillo INTERFACE)
35-
TARGET_INCLUDE_DIRECTORIES(armadillo INTERFACE ${ARMADILLO_INCLUDE_DIRS})
36-
TARGET_LINK_LIBRARIES(armadillo INTERFACE ${ARMADILLO_LIBRARIES})
3734
ENDIF ()
38-
IF (armadillo_FOUND OR ARMADILLO_FOUND OR TARGET armadillo)
39-
ADD_LIBRARY(armadillo::armadillo ALIAS armadillo)
35+
IF (NOT TARGET armadillo::armadillo)
36+
IF (TARGET armadillo)
37+
ADD_LIBRARY(armadillo::armadillo ALIAS armadillo)
38+
ELSEIF (Armadillo_FOUND)
39+
ADD_LIBRARY(armadillo INTERFACE)
40+
TARGET_INCLUDE_DIRECTORIES(armadillo INTERFACE ${ARMADILLO_INCLUDE_DIRS})
41+
TARGET_LINK_LIBRARIES(armadillo INTERFACE ${ARMADILLO_LIBRARIES})
42+
ADD_LIBRARY(armadillo::armadillo ALIAS armadillo)
43+
ENDIF ()
44+
ENDIF ()
45+
IF (TARGET armadillo::armadillo)
4046
LIST(APPEND carma_INCLUDE_DIRS ${ARMADILLO_INCLUDE_DIRS})
4147
ENDIF ()
4248

0 commit comments

Comments
 (0)