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
leap-motion: init at 2.3.1 #70522
leap-motion: init at 2.3.1 #70522
Conversation
7e09980
to
ce9a0b9
Compare
ce9a0b9
to
4eb5df7
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.
Awesome, works decently with my ancient leap motion! The binaries seem to have some weird occasional issues, but that's probably fine for a start.
cp ./usr/bin/Recalibrate $out/bin | ||
cp ./usr/bin/Visualizer $out/bin | ||
|
||
cp ./usr/bin/Playground $out/ |
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.
This should probably go to $out/bin
as well?
wantedBy = [ "default.target" ]; | ||
|
||
serviceConfig = { | ||
Type = "simple"; |
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.
This is the default, no need to specify it
Type = "simple"; | ||
Restart = "always"; | ||
ExecStart = "${pkgs.leap-motion}/bin/leapd"; | ||
KillSignal = "SIGKILL"; |
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.
Like this the service is running as root
which is not that great. Try using DynamicUser = true
instead
Thank you for your contributions.
|
meta = { | ||
homepage = https://www.leapmotion.com/; | ||
description = "Core software for the Leap Motion"; | ||
license = stdenv.lib.licenses.unfree; | ||
platforms = stdenv.lib.platforms.linux; | ||
maintainers = [ stdenv.lib.maintainers.noneucat ]; | ||
}; | ||
|
||
} |
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.
meta = { | |
homepage = https://www.leapmotion.com/; | |
description = "Core software for the Leap Motion"; | |
license = stdenv.lib.licenses.unfree; | |
platforms = stdenv.lib.platforms.linux; | |
maintainers = [ stdenv.lib.maintainers.noneucat ]; | |
}; | |
} | |
meta = with lib; { | |
homepage = https://www.leapmotion.com/; | |
description = "Core software for the Leap Motion"; | |
license = licenses.unfree; | |
platforms = platforms.linux; | |
maintainers = [ maintainers.noneucat ]; | |
}; | |
} |
Closing due to inactivity. Feel free to reopen the discussion. |
Motivation for this change
Adds support for the Leap Motion controller.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)