Conversation
|
@blowekamp please take a look 🙏 . I kept the HelloWorld as simple as possible for people getting started. |
dzenanz
left a comment
There was a problem hiding this comment.
Some remarks inline. Brad should review too.
Includes ITK CMake target configuration updates.
For simplicity we just use ITK_INTERFACE_LIBRARIES.
Now on GitHub and ReadTheDocs.
From the Configuring and Building ITK section. These are passed through in the minted code listing.
…ation() Approach with ITK 6 CMake targets.
blowekamp
left a comment
There was a problem hiding this comment.
Thank you for updating this document. A couple suggestion to improve clarity and correctness.
|
This is probably a number PR, but I have found using FetchContent with ITK very useful and much easier that an Superbuild style external project. Just a few lines are needed to fetch and build as part of your project and all the CMake configuration is inherited by ITK. My initial example is here: https://github.com/InsightSoftwareConsortium/ITK/tree/main/Examples/HelloFetchITK |
Demonstrates relationship between ITKCommon module and ITK::ITKCommonModule target.
…on() Also make sure to use ITKImageIO for the find_package COMPONENT.
|
@blowekamp thanks for the review! I agree that the FetchContent approach is better than ExternalProject for many use cases. And the HelloFetchITK is a great contribution. But I'll leave incorporating that into this documentation for a different PR. |
No description provided.