View Issue Details [ Jump to Notes ] [ Print ]
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 
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.
TagsNo tags attached.
ProjectTBD
Typeincorrect functionality
Attached Files

 Relationships

  Notes
(0031056)
Dave DeMarle (administrator)
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 (administrator)
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

 Issue History
Date Modified Username Field Change
2013-02-02 07:30 Andrzej Giniewicz New Issue
2013-06-25 13:37 Dave DeMarle Note Added: 0031056
2013-12-27 22:24 Dave DeMarle Note Added: 0032079
2013-12-27 22:24 Dave DeMarle Assigned To => Dave DeMarle
2013-12-27 22:24 Dave DeMarle Status backlog => gerrit review
2014-09-30 10:33 Dave DeMarle Status gerrit review => closed
2014-09-30 10:33 Dave DeMarle Resolution open => fixed

Copyright © 2000 - 2018 MantisBT Team