Skip to content

Revert "more workarounds for nested swift-builds"#744

Open
ktoso wants to merge 1 commit into
mainfrom
revert-nested-swift-build-workarounds
Open

Revert "more workarounds for nested swift-builds"#744
ktoso wants to merge 1 commit into
mainfrom
revert-nested-swift-build-workarounds

Conversation

@ktoso
Copy link
Copy Markdown
Collaborator

@ktoso ktoso commented May 14, 2026

This reverts commit 06947f1.

This PR is to see if the issue is resolved in more recent nightlies.

@marcprux
Copy link
Copy Markdown
Contributor

We had the same issue with Skip's invocation of Android builds.

We addressed this by filtering everything out of the environment except for a pre-approved-list of variables when running from Xcode (as determined by checking the XCODE_VERSION_MAJOR environment variable).

In addition to filtering out SDKROOT, this can also be important for ensuring that incremental builds work, since some of Xcode's ever-changing environment variables (like LLBUILD_TASK_ID) will cause Swift to think that a complete rebuild is necessary every time. I'm not sure if this is still true with swiftbuild, but it was leading to very slow rebuilds for us.

@ktoso
Copy link
Copy Markdown
Collaborator Author

ktoso commented May 15, 2026

Hah, interesting, thanks for the info Marc! I wonder then if we just need t keep this...

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