Skip to content

[5.5] Make the lookup of the PackageDescription dylib fall back on the versioned location if needed #3567

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 the 5.5 nomination of #3560.

This makes the use of the Manifest API directory (the directory in which PackageDescription.dylib and its interface is located) more robust, so that it works with mismatching toolchains. This is a little tricky to unit-test from inside the codebase, since it involves having an old toolchain compared with the expected one. Existing unit tests make sure there are no regressions.

(cherry picked from commit 7e0f5e6)

…e versioned location if needed (swiftlang#3560)

This makes the use of the Manifest API directory (the directory in which PackageDescription.dylib and its interface is located) more robust, so that it works with mismatching toolchains.  This is a little tricky to unit-test from inside the codebase, since it involves having an old toolchain compared with the expected one.  Existing unit tests make sure there are no regressions.

(cherry picked from commit 7e0f5e6)
@abertelrud
Copy link
Contributor Author

@swift-ci please test

@abertelrud abertelrud added 5.5 ready Author believes the PR is ready to be merged & any feedback has been addressed labels Jun 21, 2021
@abertelrud abertelrud self-assigned this Jun 21, 2021
@abertelrud
Copy link
Contributor Author

@swift-ci please test macos

@abertelrud abertelrud merged commit 27bb7db into swiftlang:release/5.5 Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Author believes the PR is ready to be merged & any feedback has been addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants