Skip to content
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

Remove unused runtime library path on non-Darwin platforms #1374

Merged
merged 1 commit into from Jun 16, 2023

Conversation

finagolfin
Copy link
Contributor

finagolfin added a commit to finagolfin/swift that referenced this pull request Jun 11, 2023
Translation of apple/swift-driver#1374 to the legacy C++ Driver, with the
addition of a test.
@finagolfin
Copy link
Contributor Author

@stevapple and @kateinoigakukun, please let me know if this is okay on Windows and Wasm too.

@finagolfin
Copy link
Contributor Author

@artemcm, this is the equivalent of my C++ Driver pull you just approved, would you test and merge both?

@artemcm
Copy link
Contributor

artemcm commented Jun 12, 2023

@swift-ci test

@finagolfin
Copy link
Contributor Author

@artemcm, this one and the legacy driver backport linked above have passed all tests, just need you to approve and merge both.

@finagolfin
Copy link
Contributor Author

@artemcm, the upstream version of this pull that you approved has been merged, just need you to approve and merge this one too.

@artemcm
Copy link
Contributor

artemcm commented Jun 16, 2023

@swift-ci test

@artemcm
Copy link
Contributor

artemcm commented Jun 16, 2023

@swift-ci test Windows platform

@artemcm artemcm merged commit 261e7b7 into apple:main Jun 16, 2023
3 checks passed
@finagolfin finagolfin deleted the run branch June 17, 2023 05:13
finagolfin added a commit to finagolfin/swift that referenced this pull request Jun 19, 2023
Translation of apple/swift-driver#1374 to the legacy C++ Driver, with the
addition of a test.
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.

Specifying a SDK with -sdk /path has the driver looking in a wrong resource directory path too on non-Darwin
4 participants