New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
virt-manager-qt: Add patch to rename variables #99383
Conversation
KDE/krdc@56a0222 renamed some variables resulting in a build failure. The added patch renames the variables in virt-manager-qt accordingly.
@GrahamcOfBorg build virt-manager-qt |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
/marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
@fadenb will backport to 20.09. Btw would it make sense to file a PR against upstream with your patch? :) |
Hmm, I cherry-picked this onto my local
|
Interesting. Will check that out. |
I can confirm the issue when cherry-picking the change on top of the current release-20.09. I will try to track down the difference in 20.09 and master but that might take me a while. |
I bisected release-20.09 with the commit cherry-picked on top until arrived at ce4eb0b
|
KDE/krdc@56a0222 renamed some
variables resulting in a build failure. The added patch renames the
variables in virt-manager-qt accordingly.
Motivation for this change
Fix broken build
ZHF: #97479
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)