MantisBT - VTK
View Issue Details
0015065VTK(No Category)public2014-10-22 17:012015-02-11 14:18
Orion Poplawski 
Ben Boeckel (Kitware) 
normalminorhave not tried
closedfixed 
6.0.0 
6.2.0 
TBD
incorrect functionality
0015065: Do not use get_filename_component to determine VTK_INSTALL_PREFIX
On systems with /lib64 -> /usr/lib64 (Fedora UsrMove), VTKConfig.cmake incorrectly set VTK_INSTALL_PREFIX to "/", resulting in "//include/vtk" in VTK_INCLUDE_DIRS. VTK_INSTALL_PREFIX is known at compile/install time and should be set accordingly.
No tags attached.
Issue History
2014-10-22 17:01Orion PoplawskiNew Issue
2015-02-03 18:00Orion PoplawskiNote Added: 0034148
2015-02-03 18:33Orion PoplawskiNote Added: 0034150
2015-02-05 12:12Dave DeMarleAssigned To => Ben Boeckel (Kitware)
2015-02-05 12:13Dave DeMarleNote Added: 0034163
2015-02-09 15:54Ben Boeckel (Kitware)Note Added: 0034198
2015-02-09 15:54Ben Boeckel (Kitware)Statusbacklog => gerrit review
2015-02-11 10:41Brad KingNote Added: 0034207
2015-02-11 14:18Ben Boeckel (Kitware)Note Added: 0034210
2015-02-11 14:18Ben Boeckel (Kitware)Statusgerrit review => closed
2015-02-11 14:18Ben Boeckel (Kitware)Resolutionopen => fixed
2015-02-11 14:18Ben Boeckel (Kitware)Fixed in Version => 6.2.0

Notes
(0034148)
Orion Poplawski   
2015-02-03 18:00   
This appears to be fixed in VTK git and can probably be closed.
(0034150)
Orion Poplawski   
2015-02-03 18:33   
Sorry, my mistake, this is still an issue:

CMake/vtkModuleTop.cmake:
# Generate VTKConfig.cmake for the install tree.
set(VTK_CONFIG_CODE "
# Compute the installation prefix from this VTKConfig.cmake file location.
get_filename_component(VTK_INSTALL_PREFIX \"\${CMAKE_CURRENT_LIST_FILE}\" PATH)")
# Construct the proper number of get_filename_component(... PATH)
# calls to compute the installation prefix.
string(REGEX REPLACE "/" ";" _count "${VTK_INSTALL_PACKAGE_DIR}")
foreach(p ${_count})
  set(VTK_CONFIG_CODE "${VTK_CONFIG_CODE}
get_filename_component(VTK_INSTALL_PREFIX \"\${VTK_INSTALL_PREFIX}\" PATH)")
endforeach()
(0034163)
Dave DeMarle   
2015-02-05 12:13   
@Ben, please take a look and see if this is as trivial to fix as it looks and decide on 6.2 or 6.3 target.
(0034198)
Ben Boeckel (Kitware)   
2015-02-09 15:54   
http://review.source.kitware.com/#/t/5446 [^]
(0034207)
Brad King   
2015-02-11 10:41   
VTK should use the approach that CMake does for install(EXPORT) here:

 http://www.cmake.org/gitweb?p=cmake.git;a=blob;f=Source/cmExportInstallFileGenerator.cxx;hb=v3.1.2#l72 [^]

Basically it saves knowledge of the original install prefix, and if the current location is just a symlink to that, it uses the original instead of the symlinked path.
(0034210)
Ben Boeckel (Kitware)   
2015-02-11 14:18   
Merged.