Skip to content

[5.7] SwiftPM does not build all dependency executables before invoking plugin build command (#5636) #5642

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

abertelrud
Copy link
Contributor

This is a 5.7 nomination of #5636.

…ng plugin build command (swiftlang#5636)

Make the list of input dependencies of an llbuild command that runs a plugin-provided build tool also have dependencies on the paths of all the tools on which the plugin has declared a dependency, not just the one that appears as an executable in the build command.

Also adds a mildly elaborate unit test to make sure this works.

rdar://93679015
(cherry picked from commit a6ac80f)
@abertelrud abertelrud self-assigned this Jul 5, 2022
@abertelrud abertelrud requested a review from elsh as a code owner July 5, 2022 18:30
@abertelrud abertelrud added the 5.7 label Jul 5, 2022
@abertelrud
Copy link
Contributor Author

@swift-ci please smoke test

@abertelrud abertelrud merged commit a8ea520 into swiftlang:release/5.7 Jul 5, 2022
@abertelrud abertelrud deleted the eng/95759599-plugin-executable-dependencies-not-being-built-5.7 branch July 5, 2022 22:21
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