-
Notifications
You must be signed in to change notification settings - Fork 2
[ENH][pacakges] Enable installation of development files #57
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
Conversation
RafaelPalomar
commented
May 13, 2025
- This introduces changes to pacakge definitions and new patches to install development files. The motivation is to enable the development of 3D Slicer extensions
46f8319 to
453297e
Compare
|
LGTM |
@asiangoldfish. Should we merge it, instead of closing it? There are also conflicts with dev that should be resolved. |
|
@RafaelPalomar I'll test this with #58 together one more time with @oeramo and make sure they work together. |
|
I had a go with merging #58 into this branch locally to build OpenIGTLinkIF and open it in Slicer. Slicer isn't able to load the module and show it in the dropdown list with all the other modules. I do not know if this is related to Slicer's installation files or OpenIGTLinkIF. OpenIGTLinkIF's shared object files are in the active Guix' profile's location that is recognised by the AppLauncher, specifically [LibraryPaths]
1\path=<APPLAUNCHER_SETTINGS_DIR>/../bin
2\path=<APPLAUNCHER_SETTINGS_DIR>/../lib/Slicer-5.8
3\path=<APPLAUNCHER_SETTINGS_DIR>/../lib/Slicer-5.8/cli-modules
4\path=<APPLAUNCHER_SETTINGS_DIR>/../lib/Slicer-5.8/qt-loadable-modules
5\path=../lib/Slicer-5.8/qt-loadable-modules
size=5I used ldd on one of the shared objects, namely libqSlicerOpenIGTLinkIFModule.so, to try and check if the runtime can find the required libraries and got the following output: The libraries that were not found should be from Slicer itself. |
- This introduces changes to pacakge definitions and new patches to install development files. The motivation is to enable the development of 3D Slicer extensions
453297e to
2b79899
Compare
For reference. The aim of this PR is to enable installation of the development files. These considerations are not in scope of this PR |