Skip to content

Conversation

@3405691582
Copy link
Member

@3405691582 3405691582 commented Sep 2, 2025

While we use versioned subdirectory triples for Swift module paths, this part of the rpath should be unversioned, if at the very least, for consistency.

While we are here, ensure that -z origin is used when linking, which is necessary for ORIGIN processing.

@3405691582
Copy link
Member Author

@swift-ci please test.

@3405691582 3405691582 force-pushed the rpath branch 2 times, most recently from 05c1342 to f1cfd0d Compare September 13, 2025 19:34
While we use versioned subdirectory triples for Swift module paths, this
part of the rpath should be unversioned, if at the very least, for
consistency.

While we are here, ensure that -z origin is used when linking, which is
necessary for ORIGIN processing.
@3405691582
Copy link
Member Author

@swift-ci please test.

@3405691582
Copy link
Member Author

@swift-ci please test Windows platform.

@3405691582 3405691582 requested a review from etcwilde October 20, 2025 15:22
@3405691582 3405691582 requested a review from artemcm November 10, 2025 18:52
@3405691582
Copy link
Member Author

@artemcm I'm hoping to get this in before the 6.3 cut; do you mind reviewing?

Copy link
Contributor

@owenv owenv left a comment

Choose a reason for hiding this comment

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

This LGTM

@3405691582
Copy link
Member Author

Thank you, all!

@3405691582 3405691582 merged commit 46faeab into swiftlang:main Nov 11, 2025
3 checks passed
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