Skip to content

Conversation

erneestoc
Copy link

As titled. There's an issue when remote download minimal is generating empty app bundles when using Xcode.

error: Failed to read Info.plist of app /Users/ernesto/Library/Developer/Xcode/DerivedData/InstacartBazel-eqgbkfaxbhrobdfmvkzzkbbafnzs/Build/Products/Debug-iphonesimulator/bazel-out/ios_sim_arm64-dbg-ios-sim_arm64-min16.0-applebin_ios-ST-1de1fcec1e3b/bin/Instacart.app: The operation couldn't be completed. (SWBUtil.PropertyListConversionError error 1.) (in target 'InstacartDev' from project 'InstacartBazel')

@erneestoc erneestoc requested a review from a team as a code owner August 11, 2025 19:32
@luispadron
Copy link
Contributor

Don't think we can set this globally yet as not every remote cache supports BwoB. We use all for example for local devs

@brentleyjones
Copy link
Contributor

Don't think we can set this globally yet as not every remote cache supports BwoB.

The default for Bazel is this, so I think we are safe re-enforcing the default? Then anyone that needs to override Bazel's default will need to do it for rules_xcodeproj as well? At this point remote caches really should support this, and it's more likely that someone wants this default than accidentally breaking their Xcode builds?

Co-authored-by: Aaron Sky <[email protected]>
Signed-off-by: Ernesto Cambuston <[email protected]>
@erneestoc
Copy link
Author

and it's more likely that someone wants this default than accidentally breaking their Xcode builds?

This is what I thought as well

Copy link
Contributor

@luispadron luispadron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overriding (kinda forgot this was possible) and matching the defaults in Bazel make sense

@brentleyjones brentleyjones enabled auto-merge (squash) August 19, 2025 14:55
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.

4 participants