Skip to content

Disable Bitcode when using xcodeproj.mk#626

Merged
kirb merged 4 commits into
theos:masterfrom
elihwyma:patch-2
Jan 9, 2022
Merged

Disable Bitcode when using xcodeproj.mk#626
kirb merged 4 commits into
theos:masterfrom
elihwyma:patch-2

Conversation

@elihwyma
Copy link
Copy Markdown
Contributor

@elihwyma elihwyma commented Nov 7, 2021

When distributing apps anywhere other than the App Store the bitcode section of an app goes unused. This means it won't be stripped during processing on Apple's Servers. Disabling bitcode can make apps significantly smaller. Here is Zebra with Bitcode disabled (left) and enabled (right).
image

For further reference on Bitcode you can read this help article from Apple.

When distributing apps anywhere other than the App Store the bitcode section of an app goes unused. This means it won't be stripped during processing on Apple's Servers. Disabling bitcode can make apps significantly smaller. [Here](https://cdn.discordapp.com/attachments/688124600269144162/906961066133684234/unknown.png) is Zebra with Bitcode disabled (left) and enabled (right).
@kabiroberai
Copy link
Copy Markdown
Member

Good suggestion; can we make this configurable? The default being NO makes sense, but I can think of use cases where the user might want to override it, for example if you're using xcodeproj.mk to build an xcframework for distribution.

@elihwyma
Copy link
Copy Markdown
Contributor Author

elihwyma commented Nov 8, 2021

That makes sense. On that same note it may also be worth making ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES a configurable option

@elihwyma
Copy link
Copy Markdown
Contributor Author

elihwyma commented Jan 8, 2022

Finally got round to updating this, now by default Bitcode will be disabled however it can be enabled by doing
ExampleApp_ENABLE_BITCODE = YES

@kirb kirb merged commit 00770c0 into theos:master Jan 9, 2022
@elihwyma elihwyma deleted the patch-2 branch January 9, 2022 02:04
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