Skip to content

COMP: Update examples to the new build system to avoid warnings#241

Merged
dzenanz merged 2 commits intoInsightSoftwareConsortium:mainfrom
dzenanz:itk6
Mar 12, 2026
Merged

COMP: Update examples to the new build system to avoid warnings#241
dzenanz merged 2 commits intoInsightSoftwareConsortium:mainfrom
dzenanz:itk6

Conversation

@dzenanz
Copy link
Copy Markdown
Member

@dzenanz dzenanz commented Mar 11, 2026

Warning that was triggered:

CMake Warning (dev) at CMake/UseITK.cmake:28 (message):
  CMake/UseITK.cmake is deprecated.

  Please update CMakeLists.txt to use `find_package(ITK)` and link to
  "${ITK_INTERFACE_LIBRARIES}".

Call Stack (most recent call first):
  Modules/Remote/Montage/examples/CMakeLists.txt:23 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.

@dzenanz dzenanz requested a review from blowekamp March 11, 2026 20:43
@dzenanz
Copy link
Copy Markdown
Member Author

dzenanz commented Mar 11, 2026

@blowekamp Trying to build locally in C:/Dev/ITK-vs26 by setting Module_Montage_GIT_TAG=a94b01789bf21add8eed68773ae44ea3f864c7b7 and BUILD_EXAMPLES=ON, I run into:

CMake Error at CMake/ITKModuleAPI.cmake:87 (message):
  No such module: "Montage"
Call Stack (most recent call first):
  CMake/ITKModuleAPI.cmake:31 (itk_module_load)
  CMake/ITKModuleAPI.cmake:148 (_itk_module_config_recurse)
  C:/Dev/ITK-vs26-rider/ITKConfig.cmake:94 (itk_module_config)
  Modules/Remote/Montage/examples/CMakeLists.txt:16 (find_package)

Manually setting ITK_DIR to C:/Dev/ITK-vs26 gets overwritten by C:/Dev/ITK-vs26-rider upon clicking "Configure".

Warning that was triggered:

CMake Warning (dev) at CMake/UseITK.cmake:28 (message):
  CMake/UseITK.cmake is deprecated.

  Please update CMakeLists.txt to use `find_package(ITK)` and link to
  "${ITK_INTERFACE_LIBRARIES}".

Call Stack (most recent call first):
  Modules/Remote/Montage/examples/CMakeLists.txt:23 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.
@dzenanz dzenanz marked this pull request as ready for review March 12, 2026 13:33
Copy link
Copy Markdown
Member

@blowekamp blowekamp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dzenanz dzenanz merged commit b4ade04 into InsightSoftwareConsortium:main Mar 12, 2026
23 checks passed
@dzenanz dzenanz deleted the itk6 branch March 12, 2026 22:48
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.

3 participants