-
Notifications
You must be signed in to change notification settings - Fork 428
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
bug in install.bat, produces invalid luarocks.bat and luarocks-admin.bat under certain conditions #1232
Comments
@dwenegar Looks like the issue was introduced by your PR — could you take a look at this, please? Thanks! |
bungle
added a commit
to Kong/kong
that referenced
this issue
Jan 4, 2021
bungle
added a commit
to Kong/kong
that referenced
this issue
Jan 4, 2021
kikito
pushed a commit
to Kong/kong
that referenced
this issue
Jan 6, 2021
locao
pushed a commit
to Kong/kong
that referenced
this issue
Jan 8, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The bug was introduced by commit 0451337. Now, if
vars.COMPILER_ENV_CMD == ""
, which it can very well be, the line$COMPILER_ENV_CMD >NUL
resolves to>NUL
, which is a cmd syntax error. Accordingly, luarocks.bat and luarocks-admin.bat fail.The text was updated successfully, but these errors were encountered: