Skip to content

Commit e5cc09e

Browse files
committed
[BUILD] removed last traces of old contrib finding code
1 parent ac79a6e commit e5cc09e

File tree

2 files changed

+5
-20
lines changed

2 files changed

+5
-20
lines changed

cmake/setup_lib_find_paths.cmake

Lines changed: 5 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -38,26 +38,16 @@
3838
# actual libraries are found in the CMake files of the individual componenents.
3939
#------------------------------------------------------------------------------
4040

41+
#------------------------------------------------------------------------------
42+
# emit warning if CMAKE_FIND_ROOT_PATH is used instead of CMAKE_PREFIX_PATH
4143
if(NOT "${CMAKE_FIND_ROOT_PATH}" STREQUAL "" AND "${CMAKE_PREFIX_PATH}" STREQUAL "")
4244
set(CMAKE_PREFIX_PATH "${CMAKE_FIND_ROOT_PATH}")
43-
message(STATUS "Please switch to CMAKE_PREFIX_PATH CMAKE_FIND_ROOT_PATH ")
45+
message(STATUS "Please use CMAKE_PREFIX_PATH instead of CMAKE_FIND_ROOT_PATH ")
4446
endif()
4547

48+
#------------------------------------------------------------------------------
49+
# add contrib location of source package <source>/contrib
4650
list(APPEND CMAKE_PREFIX_PATH ${OPENMS_HOST_DIRECTORY}/contrib/)
47-
# TODO: remove as soon as all libraries where switched to proper find modules
48-
set(CONTRIB_INCLUDE_DIR "" CACHE INTERNAL "contrib include dir")
49-
set(CONTRIB_LIB_DIR "" CACHE INTERNAL "contrib lib dir")
50-
foreach(CONTRIB_PATH ${CMAKE_PREFIX_PATH})
51-
list(APPEND CONTRIB_INCLUDE_DIR "${CONTRIB_PATH}/include/")
52-
list(APPEND CONTRIB_LIB_DIR "${CONTRIB_PATH}/lib/")
53-
endforeach()
54-
55-
message(STATUS "===========================================================================")
56-
message(STATUS "CMake prefix path: ${CMAKE_PREFIX_PATH}")
57-
message(STATUS "Contrib search directories: ${CONTRIB_DIR}")
58-
message(STATUS "Contrib library directories: ${CONTRIB_LIB_DIR}")
59-
message(STATUS "Contrib include directories: ${CONTRIB_INCLUDE_DIR}")
60-
message(STATUS "===========================================================================")
6151

6252
#------------------------------------------------------------------------------
6353
# Ensure Qt includes it's libs as SYSTEM

src/openms/cmake_findExternalLibs.cmake

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -134,11 +134,6 @@ endif()
134134
# Done finding contrib libraries
135135
#------------------------------------------------------------------------------
136136

137-
if(MSVC)
138-
## needed to locate libs (put this above ADD_LIBRARY() - otherwise it will not work)
139-
link_directories(${CONTRIB_LIB_DIR})
140-
endif()
141-
142137
#except for the contrib libs, prefer shared libraries
143138
if(NOT MSVC AND NOT APPLE)
144139
set(CMAKE_FIND_LIBRARY_SUFFIXES ".so;.a")

0 commit comments

Comments
 (0)