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
nixos/wooting: add xinput support #105370
Conversation
udev rules that give NixOS permission to communicate with Wooting keyboards in xinput mode | ||
''; | ||
platforms = platforms.linux; | ||
license = "unknown"; |
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.
if there's no license, I think we have to assume that it's unfree.
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.
Fixed!
# Source: https://wooting.helpscoutdocs.com/article/83-guide-configuring-xinput-support-for-the-wootings-under-linux | ||
src = [ ./wooting-xinput.rules ]; |
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.
If you already vendored the text, you could probably use a trivial builder like writeTextFile
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.
I've left the rules in a separate file for now, and have added some new rules that were added (for new keyboard models) since this PR was made.
This commit adds xinput support for Wooting keyboards to the `hardware.wooting` NixOS module, enabled by setting the `hardware.wooting.xinput.enable` option. Signed-off-by: David Wood <david@davidtw.co>
I too think there's no need (at least in the meantime) to create a package just for the udev rules. And it'd be better to add them to the service directly. Also, there's a CI error now. |
I marked this as stale due to inactivity. → More info |
Motivation for this change
This PR adds xinput support for Wooting keyboards to the
hardware.wooting
NixOS module, enabled by setting thehardware.wooting.xinput.enable
option, see upstream documentation.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)