Skip to content

[5.7] Do not test inferior Swift libraries on Darwin #5614

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

neonichu
Copy link
Contributor

5.7 cherry-pick of #5611

This is needed on all active branches because CI will upgrade to macOS 12.

On platforms with ABI-stable Swift runtime libraries (currently only Darwin), we should not set `DYLD_LIBRARY_PATHS` to include the inferior Swift libs in `bootstrap` because compatibility issues can arise from SwiftPM's use of Foundation (which itself is a client of the stdlib) and from using dyld4 (which is used on macOS 12+).

rdar://88475529

(cherry picked from commit d045aa6)
@neonichu neonichu added the 5.7 label Jun 22, 2022
@neonichu neonichu self-assigned this Jun 22, 2022
@neonichu
Copy link
Contributor Author

@swift-ci please smoke test

@neonichu
Copy link
Contributor Author

neonichu commented Nov 1, 2022

I think we can close this now.

@neonichu neonichu closed this Nov 1, 2022
@tomerd tomerd deleted the do-not-include-swift-libs-in-dyld-library-path-5.7 branch May 1, 2023 17:52
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