-
Notifications
You must be signed in to change notification settings - Fork 208
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
Add script/setup #11
Add script/setup #11
Conversation
|
||
# Enable accessibility - https://github.com/jacobsalmela/tccutil | ||
# FIXME Doesn't work with SIP https://github.com/jacobsalmela/tccutil/issues/18 | ||
# sudo tccutil -e org.hammerspoon.Hammerspoon |
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.
@MikeMcQuaid do you know any tricks to programmatically enable Accessibility?
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 do not, sorry.
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.
No problem, thanks.
@@ -0,0 +1,6 @@ | |||
cask_args appdir: '/Applications' |
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.
This is the default now so can be dropped.
ln -sf $PWD/hammerspoon ~/.hammerspoon | ||
|
||
# Prepare custom settings for navigating between words in iTerm2 | ||
cat $PWD/inputrc >> ~/.inputrc |
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.
Pedantry: may want to check the contents aren't already present to avoid appending multiple times.
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.
👍 done in be820cf
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.
Nice work, @bkeepers! 😻
I noticed just one small issue related to ~/.inputrc
. Would you mind taking a look?
ln -sfn $PWD/hammerspoon ~/.hammerspoon | ||
|
||
# Prepare custom settings for navigating between words in iTerm2 | ||
grep -q forward-word ~/.inputrc || cat $PWD/inputrc >> ~/.inputrc |
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.
If ~/.inputrc
doesn't exist, this line fails with the following error:
grep: /Users/me/.inputrc: No such file or directory
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.
👍 Fixed by adding -s
flag:
-s, --no-messages
Silent mode. Nonexistent and unreadable files are ignored (i.e.
their error messages are suppressed).
Thanks, @bkeepers! 🍻 |
This is an attempt to fix the TODO in the README for adding
script/setup
. This works on my machine, but there's still some cleanup: