Skip to content

Remove <toolchain>/usr/bin/../lib/ from directories passed to linker to look up libraries for target triple #6824

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
Aug 22, 2023

Conversation

finagolfin
Copy link
Member

Also, update my github username in the mailmap.

Resolves #6767

@neonichu, let me know what you think.

…to look up libraries for target triple

Also, update my github username in the mailmap.

Resolves swiftlang#6767
@neonichu
Copy link
Contributor

@swift-ci please smoke test

@finagolfin
Copy link
Member Author

Timeout on Windows CI when running the foundation tests is unrelated.

@neonichu
Copy link
Contributor

@swift-ci please smoke test windows

@finagolfin
Copy link
Member Author

Ping, ready to go in, I'll submit for 5.9 next.

Copy link
Member

@etcwilde etcwilde left a comment

Choose a reason for hiding this comment

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

Yes please. The thing that swiftPM is building for not necessarily the thing it's running on. Implicitly mixing those libraries can be catastrophic. Thank you.

@MaxDesiatov MaxDesiatov merged commit fab4533 into swiftlang:main Aug 22, 2023
@finagolfin finagolfin deleted the tool branch August 22, 2023 17:03
finagolfin added a commit to finagolfin/swift-package-manager that referenced this pull request Aug 22, 2023
…to look up libraries for target triple (swiftlang#6824)

Also, update my github username in the mailmap.

Resolves swiftlang#6767
finagolfin added a commit to finagolfin/swift-package-manager that referenced this pull request Aug 23, 2023
…to look up libraries for target triple (swiftlang#6824)

Also, update my github username in the mailmap.

Resolves swiftlang#6767
tomerd pushed a commit that referenced this pull request Aug 23, 2023
…to look up libraries for target triple (#6824) (#6837)

Also, update my github username in the mailmap.

Resolves #6767
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.

Don't add Toolchain.toolchainLibDir to the list of directories SwiftPM links against
4 participants