[go: up one dir, main page]

CMAKE_INSTALL_PREFIX used incorrectly

Submitted by Taylor Braun-Jones

Assigned to Nobody

Link to original bugzilla bug (#1100)
Version: 3.2

Description

The DESTINATION provided to the install command should not be prefixed with ${CMAKE_INSTALL_PREFIX} - it breaks things like CMAKE_STAGING_PREFIX (and probably other use cases as well). From cmake --help-command install:

   DESTINATION arguments specify the directory on disk to which a file  
   will be installed.  If a full path (with a leading slash or drive  
   letter) is given it is used directly.  If a relative path is given it  
   is interpreted relative to the value of CMAKE_INSTALL_PREFIX.  The  
   prefix can be relocated at install time using DESTDIR mechanism  
   explained in the CMAKE_INSTALL_PREFIX variable documentation.  

So, for example, instead of:

set(INCLUDE_INSTALL_DIR
"${CMAKE_INSTALL_PREFIX}/include/eigen3"
CACHE INTERNAL
"The directory where we install the header files (internal)"
)

It should simply be:

set(INCLUDE_INSTALL_DIR
"include/eigen3"
CACHE INTERNAL
"The directory where we install the header files (internal)"
)

Blocking

#558 (closed) #711 (closed)

Edited by Eigen Bugzilla