Skip to content

[5.5] [SR-14896] swift build --build-system xcode doesn't respect SWIFT_EXEC #3600

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

Conversation

abertelrud
Copy link
Contributor

This is the Swift 5.5 nomination of #3596, which fixes a problem that causes SWIFT_EXEC to not be used when the xcode build system is used.

The motivation for nominating this fix to 5.5 is that any SwiftPM 5.5 CI that is building packages for both x86_64 and arm64 on macOS is unintentionally using the stock compiler rather than the newly built one.

…SWIFT_EXEC` (swiftlang#3596)

When using the `xcode` build system, SwiftPM isn't passing down the result of overriding SWIFT_EXEC.  The same is true for `-toolchain` and `-sdk` but this will be fixed in a different PR.

Besides being incorrect and thus a generally something that should be fixed, this also affects SwiftPM CI, where the newly built compiler isn't being used when cross-compilation is active (since SwiftPM currently implements multi-architecture builds by switching build system to use xcbuild).

This is the Swift 5.5 nomination of this fix.

rdar://80194565
(cherry picked from commit 9aab44d)
@abertelrud abertelrud changed the base branch from main to release/5.5 July 9, 2021 01:35
@abertelrud abertelrud self-assigned this Jul 9, 2021
@abertelrud abertelrud added 5.5 ready Author believes the PR is ready to be merged & any feedback has been addressed labels Jul 9, 2021
@abertelrud
Copy link
Contributor Author

@swift-ci please test

@abertelrud abertelrud merged commit 051460e into swiftlang:release/5.5 Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Author believes the PR is ready to be merged & any feedback has been addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants