Skip to content

Conversation

@hjmjohnson
Copy link
Member

When preparing for the future with ITK by setting
ITK_FUTURE_LEGACY_REMOVE:BOOL=ON
ITK_LEGACY_REMOVEBOOL=ON

The future preferred macro should be used
│ - itkTypeMacro
│ + itkOverrideGetNameOfClassMacro

@dzenanz
Copy link
Member

dzenanz commented Jan 27, 2025

Among other things, this PR needs this: KitwareMedical/ITKStrain@517bb2e (#74)

@hjmjohnson hjmjohnson force-pushed the fix-legacy-remove branch 2 times, most recently from fe8a54b to 41a0005 Compare January 28, 2025 02:59
When preparing for the future with ITK by setting
ITK_FUTURE_LEGACY_REMOVE:BOOL=ON
ITK_LEGACY_REMOVEBOOL=ON

The future preferred macro should be used
│ -  itkTypeMacro
│ +  itkOverrideGetNameOfClassMacro
Set the default build package tags to v5.4.2
for capturing the ITKRemoteModuleBuildTestPackageAction
shared scripts.

This pulls the default configuration items needed
to build against ITK version v5.4.2.
Copy link
Member

@thewtex thewtex left a comment

Choose a reason for hiding this comment

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

💯 🥇

@thewtex thewtex merged commit cef0b6d into main Mar 10, 2025
45 checks passed
@thewtex thewtex deleted the fix-legacy-remove branch March 10, 2025 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants