Skip to content

multivnc: fix build failure; fix semver#506445

Merged
Sigmanificient merged 2 commits intoNixOS:masterfrom
drawbu:clement/fix-build-multinvc
Apr 6, 2026
Merged

multivnc: fix build failure; fix semver#506445
Sigmanificient merged 2 commits intoNixOS:masterfrom
drawbu:clement/fix-build-multinvc

Conversation

@drawbu
Copy link
Copy Markdown
Member

@drawbu drawbu commented Apr 4, 2026

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.

@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. labels Apr 4, 2026
@drawbu
Copy link
Copy Markdown
Member Author

drawbu commented Apr 4, 2026

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 506445
Commit: e5da428b0b392ec5ad74d002412c7874774ac5a8 (subsequent changes)
Merge: 42913e4f789fc6cee3efc1783f5965666646ed74

Logs: https://github.com/drawbu/nixpkgs-review-gha/actions/runs/23966931220


x86_64-linux

✅ 1 package built:
  • multivnc

aarch64-linux

✅ 1 package built:
  • multivnc

x86_64-darwin

No rebuilds


aarch64-darwin

No rebuilds

@drawbu drawbu marked this pull request as ready for review April 4, 2026 00:17
@nixpkgs-ci nixpkgs-ci bot requested a review from rhelmot April 4, 2026 00:52
the package was incorrectly set at 2.8.1 while it was actually android-2.1.8

the correct version at this commit is 0.6.0:
https://github.com/bk138/multivnc/blob/89225243412f43ba2903ffeda98af7fe1f8f4975/CMakeLists.txt#L3-L4
@drawbu drawbu force-pushed the clement/fix-build-multinvc branch from e5da428 to 81c9bb9 Compare April 4, 2026 04:37
@drawbu drawbu requested a review from Sigmanificient April 4, 2026 04:38
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. 2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". labels Apr 4, 2026
@Sigmanificient Sigmanificient changed the title multinvc: fix build failure; fix semver multivnc: fix build failure; fix semver Apr 6, 2026
Copy link
Copy Markdown
Member

@Sigmanificient Sigmanificient left a comment

Choose a reason for hiding this comment

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

Please rename in the first commit multinvc -> multivnc

the patch version of libvncserver was removing a patch required to build the
deps as it is using a deprecated cmake version, causing the build to fail
@drawbu drawbu force-pushed the clement/fix-build-multinvc branch from 81c9bb9 to 037e09f Compare April 6, 2026 02:23
@Sigmanificient Sigmanificient added this pull request to the merge queue Apr 6, 2026
@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 Apr 6, 2026
Merged via the queue into NixOS:master with commit 637bee2 Apr 6, 2026
26 checks passed
ilai-deutel pushed a commit to ilai-deutel/nixpkgs that referenced this pull request Apr 6, 2026
@drawbu drawbu deleted the clement/fix-build-multinvc branch April 6, 2026 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". 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. 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.

3 participants