Skip to content

Commit

Permalink
Fix Windows library install location
Browse files Browse the repository at this point in the history
Don't install LIBRARIES on Windows to `bin`; they belong in `lib`. (This
appears to have been done in a mistaken attempt to install the DLL's to
`bin`, which do need to be there, but RUNTIME DESTINATION already takes
care of this.)
  • Loading branch information
mwoehlke-kitware authored and jbeder committed Nov 10, 2017
1 parent 46dcdc3 commit d96826f
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions CMakeLists.txt
Expand Up @@ -258,16 +258,10 @@ endif()
###
### General install settings
###
if(WIN32)
set(_library_dir bin) # .dll are in PATH, like executables
else()
set(_library_dir lib)
endif()

set(INCLUDE_INSTALL_ROOT_DIR ${CMAKE_INSTALL_PREFIX}/include)

set(INCLUDE_INSTALL_DIR ${INCLUDE_INSTALL_ROOT_DIR}/yaml-cpp)
set(LIB_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/${_library_dir}${LIB_SUFFIX}")
set(LIB_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/lib${LIB_SUFFIX}")

set(_INSTALL_DESTINATIONS
RUNTIME DESTINATION ${CMAKE_INSTALL_PREFIX}/bin
Expand Down

0 comments on commit d96826f

Please sign in to comment.