Skip to content

BUG: Fix message about UseITK begin deprecated#5764

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:fix_use_itk_message
Feb 4, 2026
Merged

BUG: Fix message about UseITK begin deprecated#5764
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:fix_use_itk_message

Conversation

@blowekamp
Copy link
Member

Disable printing the UseITK deprecated message when ITK_LEGACY_SILENT is set. Also fix the formatting of the message, to escape variable expression.

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
  • Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

Disable printing the UseITK deprecated message when ITK_LEGACY_SILENT
is set. Also fix the formatting of the message, to escape variable
expression.
@blowekamp
Copy link
Member Author

May address #5762 too.

@github-actions github-actions bot added type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots labels Feb 3, 2026
@blowekamp blowekamp marked this pull request as ready for review February 3, 2026 22:02
@hjmjohnson hjmjohnson merged commit 5ddc558 into InsightSoftwareConsortium:main Feb 4, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants