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
Windows: force use of .exe extension. #2329
Comments
my |
My suggestion would be to put
At the top of the makefile. I think that the threat model here (based on the desire for "call" in another case). Is when someone hijacks your path and puts "mt.bat" or "rc.bat" in front of it. Of course in that case specifying .exe doesn't help, but having this stuff all macro'd away allows one to do things like
|
Wouldn't it be better to have:
That's the way most GNU makefiles do it... |
Ok landed in 5fb70ea. Thanks
Thank you for the suggestion but actually this is a windows specific makefile so we don't have to worry about exeext. |
Over Here @bagder noted (channelling Stefan Kanthak):
RDW notes. I'd suggest that$(MT) and $ (RC) would be a better way to go. Also see #2326
The text was updated successfully, but these errors were encountered: