Skip to content
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

gnome-boxes: fix USB redirection (#39129) #39133

Merged
merged 2 commits into from Apr 19, 2018
Merged

Conversation

kamidon
Copy link
Contributor

@kamidon kamidon commented Apr 19, 2018

This fixes USB redirection in gnome-boxes by ensuring that the
required PolicyKit action policy file is installed in the spice-gtk
package and adding that package to gnome-boxes propagatedUserEnvPkgs.

Motivation for this change

USB redirection stopped working for me and I needed it.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

I'm running with these modifications on my x86_64 NixOS laptop.

This fixes USB redirection in gnome-boxes by ensuring that the
required PolicyKit action policy file is installed in the spice-gtk
package and adding that package to gnome-boxes propagatedUserEnvPkgs.
@@ -24,6 +24,8 @@ in stdenv.mkDerivation rec {
meson ninja vala pkgconfig gettext itstool wrapGAppsHook gobjectIntrospection desktop-file-utils
];

propagatedUserEnvPkgs = [ spice-gtk ];
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add a comment mentioning that it is needed for installing the polkit rules

@xeji
Copy link
Contributor

xeji commented Apr 19, 2018

Thank you, looks good to me if you add the comment.

@kamidon
Copy link
Contributor Author

kamidon commented Apr 19, 2018

Thanks, comment added.

@jtojnar jtojnar merged commit 737c1c4 into NixOS:staging Apr 19, 2018
@kamidon
Copy link
Contributor Author

kamidon commented Apr 19, 2018

I really appreciate the quick turnaround on this. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: GNOME GNOME desktop environment and its underlying platform 10.rebuild-darwin: 0 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants