Skip to content

cmark-gfm: fix cmake 4 build errors#449569

Merged
LordGrimmauld merged 1 commit intoNixOS:masterfrom
joshprk:fix/cmark
Oct 7, 2025
Merged

cmark-gfm: fix cmake 4 build errors#449569
LordGrimmauld merged 1 commit intoNixOS:masterfrom
joshprk:fix/cmark

Conversation

@joshprk
Copy link
Contributor

@joshprk joshprk commented Oct 7, 2025

Fixing #445447 for cmark-gfm

Things done

  • 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. labels Oct 7, 2025
@nix-owners nix-owners bot requested a review from cyplo October 7, 2025 17:03
Copy link
Contributor

@LordGrimmauld LordGrimmauld left a comment

Choose a reason for hiding this comment

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

Built successfully on x86-64.
cmark-gfm looks inactive upstream, so probably little chance of landing an upstream fix. cmark has diverged a lot and also only does cmake min version 3.7, which is not enough.
Therefore, a substitute is reasonable. LGTM.

@LordGrimmauld
Copy link
Contributor

(Nit: 3.10 should be enough to fix these cmake issues, 3.13 is technically higher than it needs to be. But if it doesn't cause issues, i don't care too much.)

@LordGrimmauld LordGrimmauld added this pull request to the merge queue Oct 7, 2025
Merged via the queue into NixOS:master with commit bad3e00 Oct 7, 2025
29 of 33 checks passed
@Sigmanificient
Copy link
Member

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 449569
Commit: 41c53e61fa77fe16d8b483846ee52d9be2fcc8d0 (subsequent changes)
Merge: 86f93a796c062c486a44e90b90cd319f2ccc9234

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


x86_64-linux

✅ 2 packages built:
  • cmark-gfm
  • mmdoc

aarch64-linux

✅ 2 packages built:
  • cmark-gfm
  • mmdoc

x86_64-darwin (sandbox = true)

✅ 2 packages built:
  • cmark-gfm
  • mmdoc

aarch64-darwin (sandbox = true)

✅ 2 packages built:
  • cmark-gfm
  • mmdoc

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Oct 7, 2025
@joshprk joshprk deleted the fix/cmark branch October 7, 2025 23:16
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: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants