Skip to content

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Apr 16, 2024

@artemcm
Copy link
Contributor Author

artemcm commented Apr 16, 2024

@swift-ci test

@artemcm artemcm requested a review from nkcsgexi April 16, 2024 19:49
Copy link
Contributor

@nkcsgexi nkcsgexi left a comment

Choose a reason for hiding this comment

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

Thank you!

@artemcm artemcm merged commit 950fc1e into swiftlang:main Apr 25, 2024
@artemcm artemcm deleted the ResolveSymlinksForSubcommand branch April 25, 2024 21:57
@stevapple
Copy link
Contributor

I'm afraid this would break Windows support. IIRC we were using findExecutable for both cases because it can handle the executable suffix for us.

@kkebo
Copy link

kkebo commented Apr 27, 2024

@artemcm Do you plan to cherry-pick this to release/6.0?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants