-
Notifications
You must be signed in to change notification settings - Fork 261
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
Add /d to all cmd invocations to avoid autoruns. #1201
Conversation
Fixes microsoft/vcpkg#8268. |
I bet somebody depends on this behavior to not need to create a triplet/toolchain since it can be used to inject build flags via env variables. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
src/vcpkg/base/system.process.cpp
Outdated
} | ||
|
||
auto& rc_output = maybe_rc_output.value_or_exit(VCPKG_LINE_INFO); | ||
if (rc_output.exit_code != 0) | ||
{ | ||
Debug::println("Output:"); | ||
Debug::println(rc_output.output); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be cleaner if we move printing the debug output before the if statement, since we print in both cases (see line 1305).
This broke a Microsoft internal customer.