-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[Interpreter] Temporarily prefer /usr/lib/swift for Swift dylibs loaded by the interpreter #23983
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
[Interpreter] Temporarily prefer /usr/lib/swift for Swift dylibs loaded by the interpreter #23983
Conversation
@swift-ci please test |
Build failed |
@swift-ci test macOS |
Build failed |
@swift-ci please test linux |
Build failed |
@swift-ci test |
Build failed |
|
df0dd3a
to
f472c91
Compare
@swift-ci please smoke test |
1 similar comment
@swift-ci please smoke test |
…t dylibs from /usr/lib/swift. rdar://problem/46355503
…he DYLD_LIBRARY_PATH value in constructInvocation. rdar://problem/47819780
…first for Swift libraries.
9818025
to
d4063b5
Compare
@swift-ci please smoke test |
@swift-ci please smoke test OS X platform |
@swift-ci please test OS X platform |
@swift-ci please smoke test |
Build failed |
@swift-ci please test OS X platform |
Build failed |
d8c5806
to
92fa5ef
Compare
@swift-ci please smoke test and merge |
@swift-ci please test |
Build failed |
Build failed |
This is a cherry-pick of some 5.0 work plus a bit more, that's temporarily needed for 5.1. That need will hopefully go away soon, at which point I'll redo this to prefer the libraries next to the
swift
binary, and use/usr/lib/swift
as a fallback.rdar://problem/49610198