-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable installation of vtkAddon #6
Conversation
- Globbing of pattern "*.(h|txx)" does not seem to work. Globbing pattern separated in two patterns "*.h" and "*.txx" - Adding the ${PROJECT_NAME}_INSTALL_INCLUDE_DIR variable accounting for the install path for development files
- Added new variables (${PROJECT_NAME}_PYTHON_MODULE_LIB_DIR and ${PROJECT_NAME}_INSTALL_PYTHON_LIB_DIR) accounting for paths corresponding to python library modules files and python libraries files.
- Creation of vtkAddonInstallConfig.cmake.in templated file - Configuration of install config file - Installation of configuratio file - Adding a new variable ${PROJECT_NAME}_INSTALL_CMAKE_DIR to account for the project's 'installation cmake dir
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the patches 🙏
Beside of updating the commit message, looks good.
One last change:
->
|
This commit adds a copy of the CMake module vtkMacroKitPythonWrap based of Slicer/Slicer@78a71cd9a
@jcfr should we close this one ;) |
I will move forward with integrating this PR and will create an other issue to capture the fact the install tree should be relocatable. |
These changes will enable the installation of vtkAddon.
In addition, bf189a1 adds the
vtkMacroKitPythonWrap.cmake
file needed to buildvtkAddon
as a standalone project; this commit has been subject to discussion and might not be accepted in this pull as discussed in https://discourse.slicer.org/t/python-wrapping-on-the-new-external-vtkaddon/10826.The installation tree on linux with these changes is as follows:
given the following configuration: