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
libspnav: init at 2.3.0 #75866
libspnav: init at 2.3.0 #75866
Conversation
Please tell me if I should split this into multiple pull requests. |
Looking good now, only thing remaining is to clean up the history, with one initial commit for adding yourself as a maintainer, one commit for each package and one for the module |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good from my side
Can you rebase this instead of merging from master? |
I marked this as stale due to inactivity. → More info |
Still relevant |
pkgs/development/libraries/libspnav/configure-socket-path.patch
Outdated
Show resolved
Hide resolved
pkgs/development/libraries/libspnav/configure-socket-path.patch
Outdated
Show resolved
Hide resolved
pkgs/development/libraries/libspnav/configure-socket-path.patch
Outdated
Show resolved
Hide resolved
4db7376
to
710fe1a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A little nit, otherwise LGTM. I can't really test this currently, as I don't have access to the hardware currently.
@@ -0,0 +1,32 @@ | |||
{ stdenv, lib, fetchFromGitHub, libX11}: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{ stdenv, lib, fetchFromGitHub, libX11}: | |
{ stdenv, lib, fetchFromGitHub, libX11 }: |
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package blacklisted:
3 packages built:
The following issues got detected with the above build packages.
libspnav:
Please add a comment on the line above, explaining the purpose of this patch.
See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-patch-comment.md
Near pkgs/applications/misc/spacenav-cube-example/default.nix:15:3:
See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-phase-hooks.md Please add a comment on the line above, explaining the purpose of this patch.
See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-patch-comment.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
other than that LGTM
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 3 packages failed to build and already failed to build on hydra master:
|
Thanks to everyone who helped review this! |
Motivation for this change
This adds libspnav and spacenavd to use 6DOF mice, such as the 3DConnexion SpaceNavigator.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @