Skip to content

Conversation

@Daviey
Copy link
Contributor

@Daviey Daviey commented Oct 9, 2025

Fixes #450198

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

Upstream PR: mfontanini/libtins#553

Tested by building the package on Linux.

@Daviey Daviey mentioned this pull request Oct 9, 2025
3 tasks
@nixpkgs-ci nixpkgs-ci bot added 2.status: merge conflict This PR has merge conflicts with the target branch 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Oct 9, 2025
@nix-owners nix-owners bot requested a review from fdns October 9, 2025 13:16
@Daviey Daviey mentioned this pull request Oct 9, 2025
13 tasks
Patches CMakeLists.txt and CheckCXXFeatures.cmake to require CMake 3.10
instead of 2.8.x. CMake 4 doesn't support versions older than 3.5, and
versions between 3.5-3.10 are deprecated.

Upstream PR: mfontanini/libtins#553

Fixes: NixOS#450198
Signed-off-by: Dave Walker <[email protected]>
@Daviey Daviey force-pushed the fix/libtins-cmake4-build-failure branch from 7b85b2c to d96224c Compare October 9, 2025 16:07
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. and removed 2.status: merge conflict This PR has merge conflicts with the target branch 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Oct 9, 2025
@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Oct 9, 2025
@Sigmanificient
Copy link
Member

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 450277
Commit: d96224c6176c270874be5d140db61e907b06b8d9 (subsequent changes)
Merge: a652797297cbff193e1d66c99154d31042563828

Logs: https://github.com/Sigmanificient/nixpkgs-review-gha/actions/runs/18388133307


x86_64-linux

❌ 4 packages failed to build:
  • mathematica
  • mathematica-cuda
  • mathematica-webdoc
  • mathematica-webdoc-cuda
✅ 6 packages built:
  • compactor
  • dublin-traceroute
  • dublin-traceroute.dev
  • dublin-traceroute.lib
  • libtins
  • open5gs

aarch64-linux

✅ 6 packages built:
  • compactor
  • dublin-traceroute
  • dublin-traceroute.dev
  • dublin-traceroute.lib
  • libtins
  • open5gs

x86_64-darwin (sandbox = true)

✅ 3 packages built:
  • compactor
  • libtins
  • open5gs

aarch64-darwin (sandbox = true)

✅ 3 packages built:
  • compactor
  • libtins
  • open5gs

@leona-ya leona-ya added this pull request to the merge queue Oct 10, 2025
Merged via the queue into NixOS:master with commit fd665e4 Oct 10, 2025
29 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build failure: libtins-4.5

3 participants