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

Update makefile to add a build_rules #2

Merged
merged 1 commit into from
Dec 2, 2023

Conversation

jaymzh
Copy link
Contributor

@jaymzh jaymzh commented Nov 30, 2023

It would be really useful to be able to pull in core to
a repo the same way KE-complex_modifications does. It's entirely
possible to make rules that are not useful or reasonable to have
polluting the public page.

This allows one to do that and run make -C core build_rules, which
will just build the rules without trying to build the 'groups' or
webpage stuff.

Currently this is how I have my repo setup, and then I simply symlink
~/.config/karabiner/assets/<file> to <repo>/public/json/<file>.

This modification has all call build_rules, so there's absolutely
no change in workflow for anyone contributing their rules to the
primary repo.

It would be really useful to be able to pull in `core` to
a repo the same way `KE-complex_modifications` does. It's entirely
possible to make rules that are not useful or reasonable to have
polluting the public page.

This allows one to do that and run `make -C core build_rules`, which
will just build the rules without trying to build the 'groups' or
webpage stuff.

Currently this is how I have my repo setup, and then I simply symlink
`~/.config/karabiner/assets/<file>` to `<repo>/public/json/<file>`.

This modification has `all` call `build_rules`, so there's absolutely
no change in workflow for anyone contributing their rules to the
primary repo.

Signed-off-by: Phil Dibowitz <phil@ipom.com>
@tekezo tekezo merged commit afdbac0 into pqrs-org:main Dec 2, 2023
1 check passed
@tekezo
Copy link
Member

tekezo commented Dec 2, 2023

Thanks!
I also added a comment.
ab254c2

@jaymzh jaymzh deleted the makefiletweaks branch December 4, 2023 02:45
@jaymzh
Copy link
Contributor Author

jaymzh commented Dec 4, 2023

Thanks for the quick response.

And thanks for this project!

I recently got a new job which forces me to use a Mac (instead of linux), and KE has been the only thing keeping me sane. (I sponsored as well.)

@tekezo
Copy link
Member

tekezo commented Dec 4, 2023

Thank you so much!
I'm glad Karabiner-Elements has been useful to you!

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