-
Notifications
You must be signed in to change notification settings - Fork 31
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
fix(skhd): use service instead of plist #27
Conversation
cbc7d7f
to
01f52a4
Compare
01f52a4
to
e170f2a
Compare
@@ -23,70 +23,26 @@ def caveats; <<~EOS | |||
EOS | |||
end | |||
|
|||
plist_options :manual => "skhd" | |||
|
|||
if build.with? "logging" |
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.
Is this option necessary? It's not present in yabai. Happy to remove and do logging by default if desired.
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 agree, I don't think this is necessary and yabai.rb just does logging by default. You will need to also remove option "with-logging", "Redirect stdout and stderr to log files"
from line 8
Co-authored-by: jliaoh <[email protected]>
- Update environment variables - Remove root requirement - Remove update interval
Replaced by #30 |
Applying #25 to skhd as well :)
Tested via:
Ref #26