Skip to content
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

COMP: Bump to minimum CMake 3.16.3 #41

Merged

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/ITKLabelErodeDilate/ITK/CMake/ITKModuleExternal.cmake:15 (message):
  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/10184812/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/ITKLabelErodeDilate/ITK/CMake/ITKModuleExternal.cmake:15 (message):
  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/10184812/configure
@jhlegarreta
Copy link
Member Author

Build failures are unrelated.

Copy link
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

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

We should bump it to 3.22.1 eventually.

@dzenanz dzenanz merged commit f247673 into InsightSoftwareConsortium:main Feb 2, 2025
3 of 16 checks passed
@jhlegarreta jhlegarreta deleted the BumpMinReqCMakeVersion branch February 2, 2025 20:13
@jhlegarreta
Copy link
Member Author

X-ref: InsightSoftwareConsortium/ITK#4761

@jhlegarreta
Copy link
Member Author

X-ref InsightSoftwareConsortium/ITK#4761

We should bump it to 3.22.1 eventually.

I think it is becoming clear that time should be invested in having ways to maintain remotes easily, especially when it comes to applying changes that are relevant to all remotes. Otherwise, these efforts are not sustainable. X-ref: InsightSoftwareConsortium/ITK#5134 (comment), InsightSoftwareConsortium/ITK#4786

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