-
Notifications
You must be signed in to change notification settings - Fork 427
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
Update install.bat #958
base: master
Are you sure you want to change the base?
Update install.bat #958
Conversation
The problem here is that %errorlevel% is set on `exit /b ]]` and you pass it something funny. By adding rem=rem you can reuse a hack and get it still to execute correctly. You don't actually need to even set `rem=rem` you can just pass it nil, since nothing else in the script uses rem.
Codecov Report
@@ Coverage Diff @@
## master #958 +/- ##
==========================================
- Coverage 83.59% 78.94% -4.66%
==========================================
Files 90 88 -2
Lines 10696 8946 -1750
==========================================
- Hits 8941 7062 -1879
- Misses 1755 1884 +129
Continue to review full report at Codecov.
|
Also if Or even better, make it explicit and do |
Ahh thats true. Also, whats up with the dramatic code coverage declination? |
no idea, did you use the latest master branch to create this PR? |
Yes I did. I edited directly from the website. |
nvm the coverage for now, it needs updating according to previous comments |
The changes were made for rem=rem |
But it is still forcing the exit code to 0 instead of returning the result from the call into Lua |
The problem here is that %errorlevel% is set on `exit /b ]]` and you pass it something funny. By adding rem=rem you can reuse a hack and get it still to execute correctly. You don't actually need to even set `rem=rem` you can just pass it nil, since nothing else in the script uses rem.
The problem here is that %errorlevel% is set on `exit /b ]]` and you pass it something funny. By adding rem=rem you can reuse a hack and get it still to execute correctly. You don't actually need to even set `rem=rem` you can just pass it nil, since nothing else in the script uses rem.
Do we still need this? Looks like And from Lua's perspective that second added But in practice, I don't think a lot of people rely on our If the current behavior is producing a spurious error message thanks to If this is just producing a bogus errorlevel which goes mainly unnoticed, then I don't care much either way and I would just close the PR. |
The problem here is that
%errorlevel%
is set onexit /b ]]
and you pass it something funny. By addingrem=rem
you can reuse a hack and get it still to execute correctly. You don't actually need to even setrem=rem
you can just pass it nil, since nothing else in the script uses rem.