-
Notifications
You must be signed in to change notification settings - Fork 137
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
clink's installscript not supported fully correctly #161
Comments
fine, do it later. |
done, it has been renamed. |
Thank you! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello, if you use clink's new better way to install modules, so:
clink installscripts C:\path\to\z.lua\
, z.lua works mostly fine, except for one small issue. Every time you launch cmd you see this annoying error at the top:Basically what happens is that
clink installscripts
only accepts dirs not files, so you provide a path which also containstest_path.lua
file and that file doesn't work well when it's initialized in clink.The fix would be to move
test_path.lua
to a separate directory, e.g.modules
orhelpers
or something, asclink installscripts
ignores subdirectories. I'm not sure what the file does though (well, it tests the path, but I don't know under which circumstances, I can't see any references) and why it's in the root of the project.EDIT: oh, from the commit history I see it's some kind of a test/helper. So moving it shouldn't be any sort of a problem.
The text was updated successfully, but these errors were encountered: