Skip to content

Add flags to toolchain Info.plist to allow using macros from Apple's SDKs with an open-source toolchain when building in Xcode #66365

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

Merged
merged 1 commit into from
Jun 7, 2023

Conversation

owenv
Copy link
Contributor

@owenv owenv commented Jun 6, 2023

Add the required build settings overrides to the toolchain's info.plist to allow loading macros from the Xcode default toolchain

@owenv owenv requested a review from rintaro June 6, 2023 16:01
@owenv
Copy link
Contributor Author

owenv commented Jun 6, 2023

@swift-ci please build toolchain

@owenv owenv force-pushed the pr/ovoorhees/macro-settings branch from 599493d to 999bf66 Compare June 7, 2023 00:01
@owenv
Copy link
Contributor Author

owenv commented Jun 7, 2023

fixed typo

@swift-ci please build toolchain macOS

@owenv owenv force-pushed the pr/ovoorhees/macro-settings branch from 999bf66 to 5a7ba23 Compare June 7, 2023 00:27
@owenv
Copy link
Contributor Author

owenv commented Jun 7, 2023

fixed one more escaping issue

@swift-ci please build toolchain macOS

@owenv
Copy link
Contributor Author

owenv commented Jun 7, 2023

Verified that https://ci.swift.org/job/swift-PR-toolchain-macos/728/artifact/branch-main/swift-PR-66365-728-osx.tar.gz contains the correct Info.plist content, and that using macros from the SDK works in Xcode 15 beta with this toolchain selected. Also verified that the unknown frontend flags are correctly ignored when building using Xcode 14.3

@owenv
Copy link
Contributor Author

owenv commented Jun 7, 2023

@swift-ci please smoke test

@owenv
Copy link
Contributor Author

owenv commented Jun 7, 2023

Adding @shahmishal since this affects the swift.org toolchains

@owenv owenv requested a review from shahmishal June 7, 2023 18:12
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.

2 participants