Skip to content

Conversation

@Daviey
Copy link

@Daviey Daviey commented Oct 9, 2025

CMake 4 removed compatibility with CMake versions < 3.5, and versions between 3.5-3.10 are deprecated.

This change updates the minimum required version to 3.10 to ensure compatibility with modern CMake versions including CMake 4.x.

This issue is affecting NixOS users (see NixOS/nixpkgs#450274) and will likely impact other distributions as they upgrade to CMake 4.x.

The error message when using CMake 4.x:

CMake Error at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 3.5 has been removed from CMake.

  Update the VERSION argument <min> value.

CMake 4 removed compatibility with CMake versions < 3.5, and versions
between 3.5-3.10 are deprecated.

This change updates the minimum required version to 3.10 to ensure
compatibility with modern CMake versions including CMake 4.x.

This issue is affecting NixOS users (see
NixOS/nixpkgs#450274) and will likely impact
other distributions as they upgrade to CMake 4.x.
Daviey added a commit to Daviey/nixpkgs that referenced this pull request Oct 9, 2025
Replaces the postPatch workaround from NixOS#449882 with a proper patch file.
The patch approach is better because when the fix is merged upstream and
we update the version, the patch will fail to apply, giving us a clear
signal to remove it. With postPatch, the substitution would continue to
work silently even after upstream fixes it, causing us to carry
unnecessary delta forever.

Patches CMakeLists.txt to require CMake 3.10 instead of 2.8. CMake 4
doesn't support versions older than 3.5, and versions between 3.5-3.10
are deprecated.

Upstream PR: xdbob/xss-lock#3

Fixes: NixOS#450274
Signed-off-by: Dave Walker <dave@daviey.com>
work-jdannenberg pushed a commit to work-jdannenberg/nixpkgs that referenced this pull request Oct 10, 2025
Replaces the postPatch workaround from NixOS#449882 with a proper patch file.
The patch approach is better because when the fix is merged upstream and
we update the version, the patch will fail to apply, giving us a clear
signal to remove it. With postPatch, the substitution would continue to
work silently even after upstream fixes it, causing us to carry
unnecessary delta forever.

Patches CMakeLists.txt to require CMake 3.10 instead of 2.8. CMake 4
doesn't support versions older than 3.5, and versions between 3.5-3.10
are deprecated.

Upstream PR: xdbob/xss-lock#3

Fixes: NixOS#450274
Signed-off-by: Dave Walker <dave@daviey.com>
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.

1 participant