Skip to content

[Batch mode] <rdar://40225790> Merge pull request #16583 from davidungar/batch-remark #16599

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

Conversation

davidungar
Copy link
Contributor

[Batch Mode] Output a remark from the driver when in batch mode

Explanation: Output a remark when using batch mode, so user can tell.
Scope of issue: Alleviates confusion when trying batch mode.
Origination: Batch mode + quasi-pid fix (#16546)
Risk: Minimal. Xcode ignores driver output in stderr.
Reviewed by: Jordan Rose
Testing: Normal regression tests.
Radar: rdar://40225790

Resolves SR-NNNN.

[Batch Mode] Output a remark from the driver when in batch mode
@davidungar
Copy link
Contributor Author

@swift-ci please test

@davidungar
Copy link
Contributor Author

@swift-ci please nominate

@davidungar davidungar closed this May 14, 2018
@davidungar davidungar deleted the rdar-40225790-batch-mode-remark-swift-4.2-4-30-2018-branch branch May 14, 2018 20:11
@davidungar davidungar restored the rdar-40225790-batch-mode-remark-swift-4.2-4-30-2018-branch branch May 14, 2018 20:56
@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 01b1e21

@davidungar davidungar reopened this May 14, 2018
@davidungar
Copy link
Contributor Author

@swift-ci please test OS X platform

@davidungar
Copy link
Contributor Author

@swift-ci please nominate

@davidungar davidungar merged commit 0b0a14c into swiftlang:swift-4.2-branch-04-30-2018 May 15, 2018
@davidungar davidungar deleted the rdar-40225790-batch-mode-remark-swift-4.2-4-30-2018-branch branch June 20, 2019 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants