Skip to content

Conversation

@blowekamp
Copy link
Member

No description provided.

@blowekamp blowekamp marked this pull request as ready for review June 25, 2024 17:49
@blowekamp blowekamp requested review from jcfr and zivy June 25, 2024 17:50
@blowekamp blowekamp marked this pull request as draft July 5, 2024 12:00
@blowekamp
Copy link
Member Author

@jcfr Let me know if these changes should be integrated or fixed up for improved Slicer compatibility.

@jcfr
Copy link
Contributor

jcfr commented Jan 23, 2025

For reference, see Slicer/Slicer@fb4342c in which the corresponding patch was integrated in Slicer/SuperBuild/External_SimpleITK.cmake through the following pull request:

Copy link
Contributor

@jcfr jcfr left a comment

Choose a reason for hiding this comment

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

Those match the patches currently integrated in the Slicer fork of SimpleITK. See Slicer/SimpleITK@1236005...4328a91

@blowekamp blowekamp marked this pull request as ready for review January 23, 2025 14:52
@blowekamp
Copy link
Member Author

I am going to rebase these changes onto the current release branch.

jcfr added 2 commits January 23, 2025 09:55
Fixes the following error:

```
CMake Error at /path/to/cmake-3.22.2-linux-x86_64/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Python (missing: Interpreter Development.Module)
Call Stack (most recent call first):
  /path/to/cmake-3.22.2-linux-x86_64/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
  /path/to/cmake-3.22.2-linux-x86_64/share/cmake-3.22/Modules/FindPython.cmake:561 (find_package_handle_standard_args)
  Wrapping/Python/CMakeLists.txt:7 (find_package)
```
@blowekamp blowekamp merged commit fe5545f into SimpleITK:master Jan 23, 2025
8 checks passed
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