Skip to content

[build] Support installing swift-driver without swiftpm #34186

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
Oct 6, 2020

Conversation

porglezomp
Copy link
Contributor

The swift-driver Install phase knows how to build swift-driver using CMake. Allowing --install-swift-driver without also requiring --swift-driver allows installing it without requiring SwiftPM. This is a workaround for now, and we should unify the builds later.

The install-swift-driver phase knows how to build swift-driver using
CMake. Allowing only install-swift-driver without plain swift-driver
allows installing it without requiring swiftpm.
Copy link
Contributor

@artemcm artemcm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. I will be working on unifying the Build and Install flows of the swift-driver to both be CMake-based so we hopefully won't need this for long.

Copy link
Contributor

@nkcsgexi nkcsgexi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@artemcm
Copy link
Contributor

artemcm commented Oct 5, 2020

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Oct 5, 2020

Build failed
Swift Test Linux Platform
Git Sha - 8ffaf79

@artemcm
Copy link
Contributor

artemcm commented Oct 5, 2020

@swift-ci please test Linux platform

@swift-ci
Copy link
Contributor

swift-ci commented Oct 5, 2020

Build failed
Swift Test Linux Platform
Git Sha - 8ffaf79

@artemcm
Copy link
Contributor

artemcm commented Oct 5, 2020

@swift-ci please test Linux platform

2 similar comments
@artemcm
Copy link
Contributor

artemcm commented Oct 5, 2020

@swift-ci please test Linux platform

@artemcm
Copy link
Contributor

artemcm commented Oct 5, 2020

@swift-ci please test Linux platform

@artemcm
Copy link
Contributor

artemcm commented Oct 5, 2020

@swift-ci please test Windows platform

@swift-ci
Copy link
Contributor

swift-ci commented Oct 5, 2020

Build failed
Swift Test Linux Platform
Git Sha - 8ffaf79

@artemcm
Copy link
Contributor

artemcm commented Oct 5, 2020

@swift-ci please test Linux platform

@porglezomp
Copy link
Contributor Author

Might need to try it one more time

@artemcm
Copy link
Contributor

artemcm commented Oct 5, 2020

@swift-ci please test Linux platform

2 similar comments
@artemcm
Copy link
Contributor

artemcm commented Oct 6, 2020

@swift-ci please test Linux platform

@artemcm
Copy link
Contributor

artemcm commented Oct 6, 2020

@swift-ci please test Linux platform

@artemcm
Copy link
Contributor

artemcm commented Oct 6, 2020

@swift-ci please smoke test Linux platform

@nkcsgexi nkcsgexi merged commit e5b6333 into swiftlang:main Oct 6, 2020
@porglezomp porglezomp deleted the only-install-swift-driver branch October 6, 2020 21:37
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.

4 participants