Skip to content

[NFC][Build System: build-script] Re-organized the build_swift module #29243

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 16, 2020

Conversation

Rostepher
Copy link
Contributor

Re-organizes the build_swift module to maintain a similar nested structure to swift_build_support and other Python scripting that already exists in the project.

@Rostepher Rostepher requested review from compnerd and edymtt January 16, 2020 01:27
@Rostepher
Copy link
Contributor Author

@swift-ci please test

1 similar comment
@Rostepher
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 10df8153208284d81d08538ffd9ff4658ddb3bb1

@swiftlang swiftlang deleted a comment from swift-ci Jan 16, 2020
@swiftlang swiftlang deleted a comment from swift-ci Jan 16, 2020
…aintain a similar nested structure to swift_build_support and other Python scripting that already exists in the project.
@Rostepher Rostepher force-pushed the reorganize-build-swift-module branch from 10df815 to ff60592 Compare January 16, 2020 02:06
@Rostepher
Copy link
Contributor Author

@swift-ci please test and merge

@Rostepher
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 10df8153208284d81d08538ffd9ff4658ddb3bb1

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 10df8153208284d81d08538ffd9ff4658ddb3bb1

@Rostepher Rostepher merged commit a8e5a04 into swiftlang:master Jan 16, 2020
@Rostepher Rostepher deleted the reorganize-build-swift-module branch January 16, 2020 07:12
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.

3 participants