Skip to content

opendungeons: fix build with cmake4#455242

Merged
Aleksanaa merged 1 commit intoNixOS:masterfrom
iedame:cmake4/opendungeons
Nov 16, 2025
Merged

opendungeons: fix build with cmake4#455242
Aleksanaa merged 1 commit intoNixOS:masterfrom
iedame:cmake4/opendungeons

Conversation

@iedame
Copy link
Contributor

@iedame iedame commented Oct 24, 2025

depends on #455100 to build correctly, so please test with both fixes together

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: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Oct 24, 2025
@iedame iedame force-pushed the cmake4/opendungeons branch from bc971cf to b250346 Compare October 24, 2025 14:05
@Sigmanificient
Copy link
Member

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 455242
Commit: b2503469fae50cae8fbe0b7e3d06b739f7c948a8 (subsequent changes)
Merge: e9a8df73886c7584c60a676583644e616d66af90

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


x86_64-linux

❌ 1 package failed to build:
  • opendungeons

aarch64-linux

❌ 1 package failed to build:
  • opendungeons

@iedame
Copy link
Contributor Author

iedame commented Oct 29, 2025

@Sigmanificient depends on #455100

@iedame iedame added the 0.kind: ZHF Fixes Fixes during the Zero Hydra Failures (ZHF) campaign label Nov 3, 2025
Copy link
Contributor

@PlasmaPower PlasmaPower left a comment

Choose a reason for hiding this comment

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

LGTM. I've locally confirmed it fixes the opendungeons build when #455100 is merged in.

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. and removed 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Nov 15, 2025
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Nov 16, 2025
@Aleksanaa Aleksanaa added this pull request to the merge queue Nov 16, 2025
Merged via the queue into NixOS:master with commit 759194e Nov 16, 2025
34 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0.kind: ZHF Fixes Fixes during the Zero Hydra Failures (ZHF) campaign 10.rebuild-darwin: 0 This PR does not cause any packages 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. 12.approvals: 2 This PR was reviewed and approved by two persons.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants