Skip to content

Conversation

jakepetroules
Copy link
Contributor

Open Source macOS toolchains also use the .xctoolchain suffix, so this isn't the right way to detect Xcode toolchains specifically.

Open Source macOS toolchains also use the .xctoolchain suffix, so this isn't the right way to detect Xcode toolchains specifically.
@jakepetroules
Copy link
Contributor Author

@swift-ci test

@dschaefer2
Copy link
Member

I wonder if there are other .app's out there that include an OSS xctoolchain. Also, if you just have the Command Line Tools and no Xcode, you still get an Xcode toolchain, no? Where is it located? It seems you can look at the contents of the xctoolchain root plist file to get a better hint, maybe?

@jakepetroules
Copy link
Contributor Author

I wonder if there are other .app's out there that include an OSS xctoolchain.

That's a really interesting point w.r.t. possible third party IDEs, although I'm not too concerned about that at this particular point in time compared to making today's expected workflows work correctly.

Also, if you just have the Command Line Tools and no Xcode, you still get an Xcode toolchain, no? Where is it located? It seems you can look at the contents of the xctoolchain root plist file to get a better hint, maybe?

I don't think CLT will work at all with the current setup (we probably need a new enum case for it) and it hasn't been tested; we'll need to revisit it at some point.

@jakepetroules
Copy link
Contributor Author

@swift-ci test windows

1 similar comment
@jakepetroules
Copy link
Contributor Author

@swift-ci test windows

@jakepetroules jakepetroules enabled auto-merge (rebase) September 11, 2025 17:05
@jakepetroules jakepetroules merged commit 7dfca5a into swiftlang:main Sep 11, 2025
6 checks passed
@jakepetroules jakepetroules deleted the eng/PR-detect-xcode-correctly branch September 11, 2025 22:33
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