Skip to content

Conversation

@jhlegarreta
Copy link
Member

Bump to minimum CMake 3.16.3: use same cmake_minimum_required values as ITK.

Bumped in ITK in commit
InsightSoftwareConsortium/ITK@b01b325

Fixes:

-- cmake_minimum_required of 3.10.2 is not enough.
CMake Warning at D:/a/ITKNDReg/ITK/CMake/ITKModuleExternal.cmake:15 (message):
-- This is needed to allow proper setting of CMAKE_MSVC_RUNTIME_LIBRARY.
-- Do not be surprised if you run into link errors of the style:
  LNK2038: mismatch detected for 'RuntimeLibrary': value 'MTd_Static' doesn't match value 'MDd_Dynamic' in module.obj
  cmake_minimum_required must be at least 3.16.3
Call Stack (most recent call first):
  CMakeLists.txt:7 (include)

raised for example in:
https://open.cdash.org/builds/10184907/configure

Bump to minimum CMake 3.16.3: use same `cmake_minimum_required` values
as ITK.

Bumped in ITK in commit
InsightSoftwareConsortium/ITK@b01b325

Fixes:
```
-- cmake_minimum_required of 3.10.2 is not enough.
CMake Warning at D:/a/ITKNDReg/ITK/CMake/ITKModuleExternal.cmake:15 (message):
-- This is needed to allow proper setting of CMAKE_MSVC_RUNTIME_LIBRARY.
-- Do not be surprised if you run into link errors of the style:
  LNK2038: mismatch detected for 'RuntimeLibrary': value 'MTd_Static' doesn't match value 'MDd_Dynamic' in module.obj
  cmake_minimum_required must be at least 3.16.3
Call Stack (most recent call first):
  CMakeLists.txt:7 (include)
```

raised for example in:
https://open.cdash.org/builds/10184907/configure
@jhlegarreta
Copy link
Member Author

Build failures are unrelated: x-ref PR #28.

@dzenanz dzenanz merged commit f3785c0 into InsightSoftwareConsortium:main Feb 2, 2025
24 of 28 checks passed
@jhlegarreta jhlegarreta deleted the BumpMinReqCMakeVersion branch February 2, 2025 20:12
@jhlegarreta
Copy link
Member Author

X-ref: InsightSoftwareConsortium/ITK#4761

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.

2 participants