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
obinskit: init at 1.1.1 #71259
obinskit: init at 1.1.1 #71259
Conversation
@GrahamcOfBorg eval |
I've removed |
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.
Please set
dontConfigure = true;
dontBuild = true;
as the build fails trying to run these phases.
Also see my suggestion about using makeWrapper
and the use of aliases in this pr.
This is the configuration software for the enthusiast Anne Pro mechanical keyboard, used to update the firmware, change keyboard settings, clear Bluetooth connections from the device, set up backlighting colours, and so forth.
@worldofpeace thanks for the CR, I should've taken care of the feedback now, and squashed. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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.
diff LGTM
commit LGTM
app opens fine
[4 built, 2 copied (123.9 MiB), 36.1 MiB DL]
https://github.com/NixOS/nixpkgs/pull/71259
1 package built:
obinskit
Motivation for this change
This is the configuration software for the enthusiast Anne Pro mechanical keyboard, used to update the firmware, change keyboard settings, clear Bluetooth connections from the device, set up backlighting colours, and so forth. I'm an owner of one and don't think I'm alone so I thought it'd be convenient to have this packaged. They have an official Github, but the software is proprietary to my knowledge (no license provided).
I've tested that the software works with my own Anne Pro II.
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)