Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IOS-3409 - Adds Bitcode Support Back to XCode >=8.3 #201

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

IOS-3409 - Adds Bitcode Support Back to XCode >=8.3 #201

wants to merge 2 commits into from

Conversation

chipxsd
Copy link

@chipxsd chipxsd commented Apr 19, 2018

JIRA: https://layerhq.atlassian.net/browse/IOS-3409

XCode changed the default behavior when you build projects using xcodebuild which changes the arguments that get passed to the compiler and linker in the build process, specifically the ones related to the bitcode. The behavior changed in XCode v8.3.

It used to be that XCode v8.2 would implicitly compile and link with -fembed-bitcode for releases, whereas XCode versions greater than v8.2 compile and link everything with -fembed-bitcode-marker (which zeroes all the bitcode symbol maps causing the binary to include an empty Bitcode section, probably because Bitcode part of the binary is relevant only to the AppStore upload process).

But if you manually run the Build & Archive from the IDE, the code is then compiled and linked with the correct (-fembed-bitcode) argument.

We needed to get the same behavior using the xcodebuild tool. It turns out that the BITCODE_GENERATION_MODE=bitcode build setting does just that. It's normally set to marker for anything but distributable releases, which just puts and empty section (a marker) in the blob, but we need the frameworks to be distributable.

@coveralls
Copy link

coveralls commented Apr 19, 2018

Coverage Status

Coverage decreased (-2.9%) to 90.538% when pulling 2bf1d7a on layerhq:bugfix/IOS-3409-BitcodeSupport into 74343b5 on CocoaPods:master.

@chipxsd
Copy link
Author

chipxsd commented May 23, 2018

Can anyone take a look at this PR?

@segiddins
Copy link
Member

👍 with CI passing

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.

3 participants