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
dual-function-keys: init at 1.1.0 #94097
Conversation
"$(pkg-config --cflags libevdev | cut -c 3-)" | ||
''; | ||
|
||
installPhase = '' |
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.
What about installFlags = [ "DESTDIR=$(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.
Thanks! I've made this change.
The directory layout is not correct. I need to look into this.
|
Layout looks correct now.
|
@vyp Could you take a look at this PR? How do I test this PR? The NixOS configuration will look something like this:
But I'm not sure how to get my NixOS to build off a local configuration. |
Found instructions here: https://discourse.nixos.org/t/how-to-test-changes-to-a-nixos-module/2650 |
Found instructions here: https://discourse.nixos.org/t/how-to-test-changes-to-a-nixos-module/2650 I got space to work as a control key with the following configuration:
|
@svend sorry for the delay, I just wanted to let you know for now I will have time to look at this on the weekend (or earlier). |
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.
Looks like you already figured it out :)
Btw I think you could have also used a nixos overlay where you point to dual-function-keys.nix in it.
Thanks for the review!
Do you have an example on how to use overlays with a nixos configuration? |
Say you have a overlay.nix: self: super:
{
interception-tools-plugins = {
dual-function-keys = super.callPackage /path/to/dual-function-keys.nix { };
};
} In your nixos configuration.nix: {
nixpkgs.overlays = [
(import /path/to/overlay.nix)
];
services.interception-tools = {
enable = true;
plugins = [ pkgs.interception-tools-plugins.dual-function-keys ];
udevmonConfig = "...";
}
} |
Thanks for all the help with this. Could this get merged? |
@vyp Would you be able to merge this PR? I don't have commit access. |
I have also tested the the package from this PR and it worked fine for me too. |
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 squash the commits related to the package together.
pkgs/tools/inputmethods/interception-tools/dual-function-keys.nix
Outdated
Show resolved
Hide resolved
pkgs/tools/inputmethods/interception-tools/dual-function-keys.nix
Outdated
Show resolved
Hide resolved
pkgs/tools/inputmethods/interception-tools/dual-function-keys.nix
Outdated
Show resolved
Hide resolved
Result of |
64c7e1b
to
b3003cd
Compare
@svend Please resolve the merge conflict. |
b3003cd
to
802ec1b
Compare
@SuperSandro2000 I've rebased my changes and fixed the conflict. |
pkgs/tools/inputmethods/interception-tools/dual-function-keys.nix
Outdated
Show resolved
Hide resolved
pkgs/tools/inputmethods/interception-tools/dual-function-keys.nix
Outdated
Show resolved
Hide resolved
Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
Motivation for this change
Add dual-function-keys interception-tools plugin. This allows for modifier keys like: hold for ctrl, tap for delete.
See https://gitlab.com/interception/linux/plugins/dual-function-keys
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)