-
Notifications
You must be signed in to change notification settings - Fork 70
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
Installation step missing? #67
Comments
Thanks for reporting this. If you install luarocks and fennel, you should not need to compile the fennel into lua. You will need to reboot as our init.lua changes the load paths to support fennel on startup but doesn't change that on future reloads. |
I'm not sure if this is the same issue, but I:
And I get this error whenever I reload the config:
Spacehammer was working ok prior to me updating hammerspoon |
@tntmarket |
For others running into this issue, next fixed it for me: -- ...
package.path = package.path .. ";" .. "/usr/local/share/lua/5.3/?.lua"
fennel = require("fennel")
-- ... |
I did the same steps as @tntmarket and got the same error. Thanks @larryskye , adding that line to init.lua fixed it for me. |
I managed to fix this by upgrading to lua 5.4 via these steps, except I changed the last step
as otherwise it would not install fennel to ~/.luarocks. |
For me reinstall a latest hammerspoon fix the problem |
The issue seems to be that fennel was not installed into the same version of Lua that Hammerspoon is using. Make sure to use the correct one and the issue should go away. |
I wanted to try this cool project out, but it looks like a step in the installation process is missing: it looks like you need to compile the fennel files. Is that right?
What I did
I didn't have luarocks or fennel installed, so I did this:
Then I reloaded the hammerspoon config with the button in the top-right. (Note I didn't reboot.)
Expected result
Cool spacehammer keybindings.
Actual result
What resolved it for me
I added this makefile:
and ran
make all
.(One file actually filed to compile, but that didn't seem to matter.)
P.S.
Awesome project! I look forward to exploring it more.
The text was updated successfully, but these errors were encountered: