MantisBT - VTK
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0013858VTK(No Category)public2013-02-02 07:302014-09-30 10:33
ReporterAndrzej Giniewicz 
Assigned ToDave DeMarle 
PrioritylowSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
ProjectTBD
Typeincorrect functionality
Summary0013858: Default setup for Python bindings should respect $DESTDIR
DescriptionDefault install of Python bindings does not respect DESTDIR variable. On Arch Linux we apply patch along the lines of:

--- VTK/Wrapping/Python/CMakeLists.txt.orig 2013-02-02 12:56:08.067142089 +0100
+++ VTK/Wrapping/Python/CMakeLists.txt 2013-02-02 12:58:16.182794679 +0100
@@ -558,7 +558,7 @@
 
     # Create default python setup arguments if they are not set.
     IF(NOT DEFINED VTK_PYTHON_SETUP_ARGS)
- SET(VTK_PYTHON_SETUP_ARGS "--prefix=\"${DOLLAR}{CMAKE_INSTALL_PREFIX}\""
+ SET(VTK_PYTHON_SETUP_ARGS "--root=\"${DOLLAR}DESTDIR\" --prefix=\"${DOLLAR}{CMAKE_INSTALL_PREFIX}\""
         CACHE STRING "Arguments passed to \"python setup.py install ...\" during installation.")
       MARK_AS_ADVANCED(VTK_PYTHON_SETUP_ARGS)
     ENDIF(NOT DEFINED VTK_PYTHON_SETUP_ARGS)

although this isn't cross-platform solution.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-02-02 07:30Andrzej GiniewiczNew Issue
2013-06-25 13:37Dave DeMarleNote Added: 0031056
2013-12-27 22:24Dave DeMarleNote Added: 0032079
2013-12-27 22:24Dave DeMarleAssigned To => Dave DeMarle
2013-12-27 22:24Dave DeMarleStatusbacklog => gerrit review
2014-09-30 10:33Dave DeMarleStatusgerrit review => closed
2014-09-30 10:33Dave DeMarleResolutionopen => fixed

Notes
(0031056)
Dave DeMarle   
2013-06-25 13:37   
We'll have a close look at this in 6.1. We hope to get rid of setup.py and have cmake install itself the python extension modules, which should fix this and the rpath issues.
(0032079)
Dave DeMarle   
2013-12-27 22:24   
Andrzej, would you mind checking the the master branch (what will be 6.0.1 rc 1 in a few days)?
I removed setup.py from the install process a few months ago and this particular problem is likely moot now.

commit beae7c648708c1ae453d9fb8a19a5a89638379bc
Date: Thu Aug 15 11:50:53 2013 -0400