Skip to content

Remove last trace of TSCUtility #1273

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

Merged
merged 1 commit into from
Jan 27, 2023
Merged

Conversation

compnerd
Copy link
Member

Replace use of Diagnostic.fatalError with a
SwiftDriver.Diagnostics.errorsEmitted type which removes the final reference to TSCUtility in the actual shipping code. References within the test suite may remain for now.

@compnerd compnerd requested review from CodaFi and artemcm January 25, 2023 23:29
@compnerd
Copy link
Member Author

@swift-ci please test

@compnerd
Copy link
Member Author

@swift-ci please test

@compnerd
Copy link
Member Author

Ugh, this is uncovering new dependencies on TSCUtility that we didn't see before. We have a number of uses of spm_chomp and Diagnostics.emit({warning,error,note}:).

@compnerd
Copy link
Member Author

Once #1275 and #1276 are merged, this should pass.

Replace use of `Diagnostic.fatalError` with a
`SwiftDriver.Diagnostics.errorsEmitted` type which removes the final
reference to `TSCUtility` in the actual shipping code.  References
within the test suite may remain for now.
@compnerd
Copy link
Member Author

@swift-ci please test

@compnerd compnerd merged commit f4b0ef0 into swiftlang:main Jan 27, 2023
@compnerd compnerd deleted the fatally-shot branch January 27, 2023 17:57
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