Skip to content

cmtk: fix build with cmake 4#455098

Draft
samestep wants to merge 1 commit intoNixOS:masterfrom
samestep:cmtk-cmake4
Draft

cmtk: fix build with cmake 4#455098
samestep wants to merge 1 commit intoNixOS:masterfrom
samestep:cmtk-cmake4

Conversation

@samestep
Copy link
Contributor

@samestep samestep commented Oct 24, 2025

Things done

This does fix the build, but the pkgs/by-name/cm/cmtk/export-library-dependencies.patch file is AI-generated so I don't trust it (and I'm not a CMake expert so I can't review it myself).

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Oct 24, 2025
@nix-owners nix-owners bot requested a review from tbenst October 24, 2025 01:49
@samestep
Copy link
Contributor Author

Marking this as draft for now. If I have time to review the AI-generated code then I'll un-draft it.

@samestep samestep marked this pull request as draft October 24, 2025 03:17
@Sigmanificient
Copy link
Member

@samestep hello, do you have time to continue this pr? Thx!

@samestep
Copy link
Contributor Author

@Sigmanificient unfortunately no, not in the foreseeable future; I don't have sufficient CMake knowledge to confidently review or rewrite this code.

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-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants