Skip to content

[bootstrap] Remove deprecated "--build-tests" #178

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

Conversation

modocache
Copy link
Contributor

The "--build-tests" option is marked "deprecated" in the SwiftPM bootstrap script and is no longer used by the script. It is also no longer used by any systems calling the bootstrap script; its only usage in the Swift build script has been removed in swiftlang/swift#1547.

Remove the option to reduce confusion.

The "--build-tests" option is marked "deprecated" in the SwiftPM bootstrap
script and is no longer used by the script. It is also no longer used by any
systems calling the bootstrap script; its only usage in the Swift build
script has been removed in swiftlang/swift#1547.

Remove the option to reduce confusion.
@kostiakoval
Copy link
Contributor

This is trivial and safe change so I'm going to merge it.
I've tested that  ./Utilities/bootstrap test works as expected both on MacOS and Linux and we don't need "--build-tests"

kostiakoval added a commit that referenced this pull request Mar 7, 2016
[bootstrap] Remove deprecated "--build-tests"
@kostiakoval kostiakoval merged commit 0411c9c into swiftlang:master Mar 7, 2016
@modocache modocache deleted the remove-deprecated-build-tests branch March 7, 2016 18:54
@modocache
Copy link
Contributor Author

Thanks, @kostiakoval! 🙇

aciidgh pushed a commit to aciidgh/swift-package-manager that referenced this pull request Jan 11, 2019
This is a follow-up to swiftlang#158 to add the newly added delegate methods to
llbuild's C API.
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