COMP: Require ITK 6 - #482
Merged
hjmjohnson merged 1 commit intoSep 14, 2026
Merged
Conversation
github-actions
Bot
added
area:Core
hjmjohnson
marked this pull request as ready for review
September 13, 2026 17:07
dzenanz
requested changes
Sep 14, 2026
hjmjohnson
force-pushed
the
comp-require-itk6-only
branch
from
September 14, 2026 15:42
c59cf89 to
5302fe0
Compare
Every example links ITK::...Module interface targets, which ITK 5.x does not export, so against ITK 5.4 configure fails with one missing-target error per example (346 in all). Requiring ITK 6 stops at the first find_package with a single version error instead. Examples for ITK 5.x now live on the release-5.4 branch.
hjmjohnson
force-pushed
the
comp-require-itk6-only
branch
from
September 14, 2026 15:44
5302fe0 to
314ed1d
Compare
dzenanz
approved these changes
Sep 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Raise the minimum ITK to 6 in
src/CMakeLists.txt. Every example alreadylinks
ITK::<Name>Moduleinterface targets, which ITK 5.x does not export.What ITK 5.x does today, and what this changes
Against ITK 5.4 the build currently fails with one missing-target error per
example — 346 in all — because the namespaced interface targets were introduced
in ITK 6. Requiring ITK 6 at the top level stops at the first
find_packagewith a single version error instead:
Three examples still named an older minimum (5.0, 5.0, 4.7.0) from before the
CMake refactoring. They now use a bare
find_package(ITK REQUIRED)like theother 408 examples, so the requirement is stated once, in
src/CMakeLists.txt.Where ITK 5.x examples live
The
release-5.4branch was created atb0679d89, the last commit that stillbuilds against ITK 5.x. Verified against ITK 5.4.7: 0 failed targets, 346
executables, 370 of 370 tests passing.