-
Notifications
You must be signed in to change notification settings - Fork 10.5k
build: early exit on error #69582
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
build: early exit on error #69582
Conversation
@swift-ci please test |
https://ci-external.swift.org/job/swift-PR-windows/18257/console ... nope, its still silently failing. |
@swift-ci please test |
Gah, its still not reporting the error properly. |
@swift-ci please test Windows platform |
b6b8435
to
126ee47
Compare
@swift-ci please test Windows platform |
Apply more error checking to ensure that failures do not slip through the various layers of invocation.
@swift-ci please smoke test |
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.
Looks good!
If build.ps1 fails, immediately exit rather than cleaning up. We may lose the error code in such situations.