Make update_in_place opt-in (true by default)

[thiagohmcruz]

Due to the codesign issues being redirected here SwiftUI Previews will crash in Xcode (using rules_xcodeproj) with

Exception Type:  EXC_BAD_ACCESS (SIGKILL (Code Signature Invalid))

This happens only in vendored_xcframeworks that already provide the slices I need for Apple Silicon so planning to conditionally disable this update_in_place logic only in "SwiftUI Previews" builds.

More generally how are we supposed to handle this codesign errors in Debug builds where a provisioning_profile is not set? My suggestion is to allow this conditional behavior while we discuss this, happy to file an issue if needed.

[jerrymarino]

I wonder if theres arm64_sim slice why it's running it through this in the first place.

The other potential is it might fit better to just give them an ad-hoc signature

[thiagohmcruz]

I wonder if theres arm64_sim slice why it's running it through this in the first place.

@jerrymarino the internal dep I'm using to test contains these two slices in vendored_xcframeworks:

                    "identifier": "ios-arm64",
                    "platform": "ios",
                    "platform_variant": "",

                    "identifier": "ios-arm64_x86_64-simulator",
                    "platform": "ios",
                    "platform_variant": "simulator",

So IIUC it is supposed to hit this code path since import_middleman is created.

The other potential is it might fit better to just give them an ad-hoc signature

It's not clear to me how we would sign this in a generic way if it's not a device build, what would be the inputs to codesign? Also, I don't see a clear advantage in doing the ad-hoc signature vs allowing this behavior to be conditionally enabled?

[jerrymarino]

I was more of thinking since it was a platform_variant simulator it doesn't need to be called - do you see what I mean? Othwerwise could try to do a force'd ad-hoc signature sans identity.

[thiagohmcruz]

Closing in favor of #769