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

Handle macOS CLT --show-sdk-platform-path issues #6977

Merged
merged 1 commit into from Oct 11, 2023

Conversation

neonichu
Copy link
Member

@neonichu neonichu commented Oct 6, 2023

We have been seeing this issue for a while when using SwiftPM from the CommandLineTools package:

❯ /usr/bin/xcrun --sdk macosx --show-sdk-platform-path
xcrun: error: unable to lookup item 'PlatformPath' from command line tools installation
xcrun: error: unable to lookup item 'PlatformPath' in SDK '/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk'

This changes SwiftPM to handle this gracefully during builds (a warning gets emitted) and to fail properly when running swift test.

rdar://107479428

@neonichu neonichu self-assigned this Oct 6, 2023
@neonichu
Copy link
Member Author

neonichu commented Oct 6, 2023

@swift-ci please test

We have been seeing this issue for a while when using SwiftPM from the CommandLineTools package:

```
❯ /usr/bin/xcrun --sdk macosx --show-sdk-platform-path
xcrun: error: unable to lookup item 'PlatformPath' from command line tools installation
xcrun: error: unable to lookup item 'PlatformPath' in SDK '/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk'
```

This changes SwiftPM to handle this gracefully during builds (a warning gets emitted) and to fail properly when running `swift test`.

rdar://107479428
@neonichu
Copy link
Member Author

@swift-ci please test

@neonichu
Copy link
Member Author

From the Windows build:

This run spent 2 hr 28 min waiting in the queue.

😞

@neonichu
Copy link
Member Author

Looks like the Windows build succeeded now, but the status on GH hasn't been updated.

@neonichu
Copy link
Member Author

I guess there's a few minute delay, it looks green now.

@neonichu neonichu merged commit 3ab568e into main Oct 11, 2023
5 checks passed
@neonichu neonichu deleted the handle-no-PlatformPath branch October 11, 2023 03:19
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.

None yet

3 participants