Skip to content

Conversation

@thiagohmcruz
Copy link
Contributor

@thiagohmcruz thiagohmcruz commented Jan 9, 2026

Fixes #3258.

In xcodeproj/internal/files/output_files.bzl using _collect_output_files as a reference starts collecting more outputs in _collect_mixed_language_output_files to address the issue.

I'll use #3259 to verify this fix is good (I will either rebase or cherry pick it here)

Signed-off-by: Thiago Cruz <thiago@squareup.com>
@thiagohmcruz thiagohmcruz force-pushed the thiago/collect-additional-mixed-lib-output-files branch from c56229f to bf9ecc5 Compare January 9, 2026 19:21
@thiagohmcruz thiagohmcruz marked this pull request as ready for review January 9, 2026 20:09
@thiagohmcruz thiagohmcruz requested a review from a team as a code owner January 9, 2026 20:10
@thiagohmcruz
Copy link
Contributor Author

thiagohmcruz commented Jan 9, 2026

Waiting for updates on the CI issue, should be able to land this Mon!

@luispadron
Copy link
Contributor

Waiting for updates on the CI issue, should be able to land this Mon!

Closing/opening the PR triggered the jobs

@thiagohmcruz
Copy link
Contributor Author

Taking a look at the failures

@brentleyjones
Copy link
Contributor

The HEAD failures are expected. The lint one isn't.

Signed-off-by: Thiago Cruz <thiago@squareup.com>
@thiagohmcruz thiagohmcruz force-pushed the thiago/collect-additional-mixed-lib-output-files branch from e4732c3 to b83aaab Compare January 12, 2026 18:09
@thiagohmcruz
Copy link
Contributor Author

The HEAD failures are expected. The lint one isn't.

@brentleyjones yeah I TIL-ed about these failures this AM, found issues reported in Bazel for the two failures we're hitting here indeed, thx for flagging!

Just pushed a fix for the linter

@brentleyjones brentleyjones merged commit 773aff0 into main Jan 12, 2026
9 of 11 checks passed
@brentleyjones brentleyjones deleted the thiago/collect-additional-mixed-lib-output-files branch January 12, 2026 18:15
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.

Bug: Xcode build of mixed_language_library target succeeds even if source file contains breaking changes

5 participants