CTK icon indicating copy to clipboard operation
CTK copied to clipboard

Add 'rules' to install from superbuild

Open mwoehlke-kitware opened this issue 10 years ago • 3 comments

Import ExternalProject_Install_CMake from commontk/Artichoke@b0cf6fc85d5a, and use to install (some) external projects when building the install target of a CTK superbuild.

Note that this (currently) does not install DCMTK, KWStyle or QtTesting or PythonQt. KWStyle is probably not needed, but the other three should be fixed (and also, the non-installed CTK build should use their build trees rather than doing local installs). As a result, this is NOT sufficient yet to produce a usable install of CTK from the superbuild.

mwoehlke-kitware avatar May 20 '14 21:05 mwoehlke-kitware

I was looking into doing a "super-build install" of CTK as well and was wondering what is status of this pull request?

I would be willing to have a look and merge it if there are no reasons not to, even if it doesn't support all options / external projects yet. I would also work on issue #406 in parallel.

saschazelzer avatar Jan 16 '15 15:01 saschazelzer

@saschazelzer This effort hasn't been completed because instead the support of component installation within CMake itself should be updated. More details here: http://public.kitware.com/Bug/view.php?id=14955

Since updating CMake itself may require more work, it may be worth continuing with this topic ... I guess sending an email on the CMake developer list could be useful.

jcfr avatar Jan 20 '15 04:01 jcfr

Thanks for the clarification! I will send a mail to the cmake list about this topic.

Anyway, the changes look small enough to warrant their inclusion in my opinion. It will take some time until CMake supports this natively and more time until we can require a recent CMake version in CTK. I suggest that I extract only the build system modifications for "super-build install support" from this merge request and bring it in a usable state (together with fixes for issue #406).

saschazelzer avatar Jan 20 '15 10:01 saschazelzer

Closing. This pull request is obsolete.

jcfr avatar Apr 13 '23 23:04 jcfr