MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0009563ParaViewBugpublic2009-09-17 16:522011-02-18 20:29
ReporterTyson Whitehead 
Assigned ToDavid Partyka 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version3.6 
Target VersionFixed in Version3.10 
Project
Topic Name
Type
Summary0009563: The CMake file for StreamingParaview doesn't work with DESTDIR under Linux in copying the QT libraries
DescriptionDo a "make install" with "DESTDIR" set and you will get many tar "Cannot chdir: No such file or directory" error messages when it tries to copy the various QT libraries over for StreamingParaview. The problem is in the following code in Applications/StreamingParaView/CMakeLists.txt which doesn't take DESTDIR into account

IF(NOT Q_WS_MAC)
  FOREACH(qtlib ${QTLIBLIST})
    IF (NOT WIN32)
      #INSTALL(FILES ${QT_${qtlib}_LIBRARY_RELEASE} DESTINATION ${PV_INSTALL_LIB_DIR})
      GET_FILENAME_COMPONENT(QT_LIB_DIR_tmp ${QT_${qtlib}_LIBRARY_RELEASE} PATH)
      GET_FILENAME_COMPONENT(QT_LIB_NAME_tmp ${QT_${qtlib}_LIBRARY_RELEASE} NAME)
      FILE(GLOB QT_LIB_LIST RELATIVE ${QT_LIB_DIR_tmp} "${QT_${qtlib}_LIBRARY_RELEASE}*")
      INSTALL(CODE "
MESSAGE(STATUS \"Installing \${CMAKE_INSTALL_PREFIX}/${PV_INSTALL_LIB_DIR}/${QT_LIB_NAME_tmp}\")
EXECUTE_PROCESS (WORKING_DIRECTORY ${QT_LIB_DIR_tmp}
                 COMMAND tar c ${QT_LIB_LIST}
                 COMMAND tar -xC \${CMAKE_INSTALL_PREFIX}/${PV_INSTALL_LIB_DIR})
        " COMPONENT Runtime)
    ELSE (NOT WIN32)
      GET_FILENAME_COMPONENT(QT_DLL_PATH_tmp ${QT_QMAKE_EXECUTABLE} PATH)
      INSTALL(FILES ${QT_DLL_PATH_tmp}/${qtlib}4.dll DESTINATION ${PV_INSTALL_BIN_DIR} COMPONENT Runtime)
    ENDIF (NOT WIN32)

  ENDFOREACH(qtlib)
ENDIF(NOT Q_WS_MAC)

Note that the installation is not killed though, so you have to capture the output if you wish to see this error. Actually, I'm not sure I understand why we want a personal copy of these libraries in this case in the first place. Why not just use them as is?
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-09-17 16:52Tyson WhiteheadNew Issue
2009-09-17 17:25Tyson WhiteheadNote Added: 0017597
2010-11-30 22:23David PartykaAssigned To => David Partyka
2010-11-30 22:23David PartykaStatusbacklog => tabled
2011-02-17 15:23David PartykaNote Added: 0025464
2011-02-17 15:23David PartykaStatustabled => @80@
2011-02-17 15:23David PartykaFixed in Version => 3.10
2011-02-17 15:23David PartykaResolutionopen => fixed
2011-02-18 20:29Alan ScottNote Added: 0025513
2011-02-18 20:29Alan ScottStatus@80@ => closed

Notes
(0017597)
Tyson Whitehead   
2009-09-17 17:25   
I should also note that the same code appears twice is in Applications/Client/CMakeLists.txt as well.
(0025464)
David Partyka   
2011-02-17 15:23   
Streaming is no longer it's own application and is instead a native plugin of ParaView itself.
(0025513)
Alan Scott   
2011-02-18 20:29   
Not tested by closer.