-
Notifications
You must be signed in to change notification settings - Fork 119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build when ITK_LEGACY_REMOVE=ON #617
Comments
@N-Dekker Could you have a look? It seems important for itk-elastix as well. |
Indeed, my motivation was to get ITKElastix building against current master of ITK and elastix. |
@dzenanz It appears that in order to compile elastix against the current master of ITK, we would have to drop ITK 5.2 support. While ITK 5.3 is not yet released. That's why I hesitate. Do you know when ITK 5.3 is aimed to be released? For the time being, the following branch should be compatible with the current master of ITK: |
The final 5.3.0 is expected within weeks. It will be cut from master. I understand hesitation. The branch you provided will probably be enough to get ITKElastix compiling. If not, I will follow up in its repository. |
Resolved by pull request #620 "COMP: Upgrade ITK from v5.2.0 to v5.3rc03" (merged on 18 February 2022). |
This option is forced to ON when Python wrapping is enabled. If I try to build it against a recent ITK master, I get:
The text was updated successfully, but these errors were encountered: