Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[ios, build] Re-add support for 32-bit simulators (i386) #11891

Merged
merged 1 commit into from
May 11, 2018

Conversation

friedbunny
Copy link
Contributor

Reverts the removal of i386 slices from our framework in #10962/ios-v4.0.0. This is because CocoaPods effectively forces pods to package i386 in all of their dependencies (e.g., mapbox/mapbox-navigation-ios#1394).

This puts us back at the ios-v3.7.x state, where we strip i386 from dSYMs only (#10781).

/cc @bsudekum @1ec5 @akitchen @fabian-guerra

@friedbunny friedbunny added iOS Mapbox Maps SDK for iOS build CocoaPods labels May 11, 2018
@friedbunny friedbunny added this to the ios-v4.0.1 milestone May 11, 2018
@friedbunny friedbunny self-assigned this May 11, 2018
@@ -4,10 +4,14 @@ Mapbox welcomes participation and contributions from everyone. Please read [CONT

## 4.0.1

### Packaging

* Re-added support for 32-bit simulators (i386) to avoid conflicts with CocoaPods. ([#11891](https://github.com/mapbox/mapbox-gl-native/pull/11891))
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe word it more strongly, like “to work around an issue in CocoaPods”, since it doesn’t seem to be a bug on our end?

@friedbunny friedbunny force-pushed the fb-never-gonna-give-i386-up branch from 20f88d8 to 0d067e3 Compare May 11, 2018 20:38
@friedbunny friedbunny merged commit 0d067e3 into release-boba May 11, 2018
@friedbunny friedbunny deleted the fb-never-gonna-give-i386-up branch May 11, 2018 22:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
build iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants