Skip to content
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

Enables NixOS compatability #41

Closed
wants to merge 2 commits into from
Closed

Conversation

carsonbaker
Copy link

The keyd Makefile assumes the Filesystem Hierarchy Standard, which NixOS does not conform to. I wrote this patch to allow the -c flag to specify an alternative to the /etc/keyd config directory.

It looks like this option precludes the use of other flags (e.g. -d) at the same time.

I don't write C code very often -- sorry there is likely a much better way to do this.

Cheers for the great work on keyd. I appreciate it.

@carsonbaker
Copy link
Author

Sorry, I didn't actually see the following pulls/issues before I wrote this PR. Mentioning them here now:
#7
#18

@rvaiya
Copy link
Owner

rvaiya commented Oct 14, 2021

Thanks for the patch, but please open an issue next time first. Distro specific files like shell.nix should be maintained downstream. The config directory can be altered at compile time (though NixOS has /etc as noted in #7).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants