Skip to content
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

Install into appropriate platform directories #877

Closed
wants to merge 1 commit into from

Conversation

TheChymera
Copy link
Contributor

@gdevenyi
Copy link
Contributor

Thanks @TheChymera this is a good start but it doesn't modify the superbuild codepath we had trouble with in my previous PR. That's the way most people are installing ANTs still.

I think this can be addressed if you could provide some detail on how you got your ITK install to honour this lib vs lib64 setting, since we'll need to do something like that to the ITK and VTK in the superbuild.

@TheChymera
Copy link
Contributor Author

I think this can be addressed if you could provide some detail on how you got your ITK install to honour this lib vs lib64 setting, since we'll need to do something like that to the ITK and VTK in the superbuild.

This patch was all that was needed (in addition to your logic patch from #867 --- though it seems unrelated) to successfully produce and install the attached file list. Does this seem unlikely, or is anything missing?

@gdevenyi
Copy link
Contributor

As always, the exact cmake configuration determines which things are built and which things are installed. Your file list does not provide suffcient information on whether this patch works. The exact cmake config is also required.

My guess is you are not testing the SuperBuild path since you are specifying USE_SYSTEM_ITK AND USE_SYSTEM_VTK. Most users are using the SuperBuild, which (in the BUILD_SHARED_LIBS config) triggers this codepath:
https://github.com/ANTsX/ANTs/blob/master/ANTS.cmake#L178

Notice that this still hard-codes a libs directory. Please test this codepath using the ANTs-documented installation instructions.

@TheChymera
Copy link
Contributor Author

@gdevenyi could you provide a link to the install instructions you are referring to, I found no unabbiguous resource in the README.

@gdevenyi
Copy link
Contributor

@TheChymera
Copy link
Contributor Author

@gdevenyi ok, so I gave it a try and it seems to work:

build.log
install.log
mytree.txt

@gdevenyi
Copy link
Contributor

Hi,

The build you logged did not use -DBUILD_SHARED_LIBS so the shared libraries were not installed and so that codepath was not triggered.

This line needs to be updated to conform with the lib vs lib64 detection inside CMAKE
https://github.com/ANTsX/ANTs/blob/master/ANTS.cmake#L178

In addition, the ITK/VTK installs may need to be adjusted so that the staging superbuild install honours lib/lib64, details on the Gentoo ITK/VTK configuration would be helpful here to see how thats handled.

cookpa referenced this pull request Jun 26, 2020
Trying to enable installation to the correct lib dir for each platform
@cookpa
Copy link
Member

cookpa commented Oct 17, 2021

Addressed in #1062

@cookpa cookpa closed this Oct 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Install libraries in /usr/lib64 instead of /usr/lib
3 participants