Skip to content

Conversation

@MaxDesiatov
Copy link
Contributor

@MaxDesiatov MaxDesiatov commented Nov 1, 2023

Dependency of swiftlang/swift-package-manager#7021.

The -ld-path option was introduced on main in swiftlang/swift-driver#1442 and release/5.10 in swiftlang/swift-driver#1442. SwiftPM needs to detect this flag to pass options to the driver correctly, and it's suitable to do this via feature.json instead of checking for the compiler version via other means.

Partially resolves rdar://117049947.

The `-ld-path` option was introduced on `main` in swiftlang/swift-driver#1442 and 5.10 in swiftlang/swift-driver#1442. SwiftPM needs to detect this flag to pass options to the driver correctly, and it's suitable to do this via `feature.json` instead of checking for the compiler version via other means.
@MaxDesiatov MaxDesiatov added the driver options Area → compiler → legacy driver: driver options label Nov 1, 2023
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov MaxDesiatov merged commit b7d8a9b into main Nov 2, 2023
@MaxDesiatov MaxDesiatov deleted the maxd/add-ld-path-feature branch November 2, 2023 18:08
MaxDesiatov added a commit that referenced this pull request Nov 4, 2023
Dependency of swiftlang/swift-package-manager#7021.

Cherry-pick of #69586.

The `-ld-path` option was introduced on `main` in swiftlang/swift-driver#1442 and 5.10 in swiftlang/swift-driver#1442. SwiftPM needs to detect this flag to pass options to the driver correctly, and it's suitable to do this via `feature.json` instead of checking for the compiler version via other means.

Partially resolves rdar://117049947.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

driver options Area → compiler → legacy driver: driver options

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants