Skip to content

Remove bitcode support#1534

Merged
keith merged 4 commits intomasterfrom
ks/remove-all-bitcode-related-tests
May 5, 2023
Merged

Remove bitcode support#1534
keith merged 4 commits intomasterfrom
ks/remove-all-bitcode-related-tests

Conversation

@keith
Copy link
Copy Markdown
Member

@keith keith commented Jul 7, 2022

Now that Xcode 14.1 is the minimum supported to the App Store, and Xcode
14.0 dropped support for bitcode, we don't need to handle these flags
anymore.

Closes #1956

keith referenced this pull request Jul 7, 2022
https://developer.apple.com/documentation/xcode-release-notes/xcode-14-release-notes

> Starting with Xcode 14, bitcode is no longer required for watchOS and tvOS applications, and the App Store no longer accepts bitcode submissions from Xcode 14.
>
> Xcode no longer builds bitcode by default and generates a warning message if a project explicitly enables bitcode: “Building with bitcode is deprecated. Please update your project and/or target settings to disable bitcode.” The capability to build with bitcode will be removed in a future Xcode release. IPAs that contain bitcode will have the bitcode stripped before being submitted to the App Store. Debug symbols for past bitcode submissions remain available for download. (86118779)

RELNOTES: None
PiperOrigin-RevId: 453277394
thomasvl and others added 3 commits May 4, 2023 08:57
https://developer.apple.com/documentation/xcode-release-notes/xcode-14-release-notes

> Starting with Xcode 14, bitcode is no longer required for watchOS and tvOS applications, and the App Store no longer accepts bitcode submissions from Xcode 14.
>
> Xcode no longer builds bitcode by default and generates a warning message if a project explicitly enables bitcode: “Building with bitcode is deprecated. Please update your project and/or target settings to disable bitcode.” The capability to build with bitcode will be removed in a future Xcode release. IPAs that contain bitcode will have the bitcode stripped before being submitted to the App Store. Debug symbols for past bitcode submissions remain available for download. (86118779)

RELNOTES: None
PiperOrigin-RevId: 453277394
(cherry picked from commit 14be0fd)
Bitcode is no longer supported for Xcode 14 and later. App Store support for uploads from Xcode prior to 14.1 will no longer be allowed after April 25.

PiperOrigin-RevId: 527298659
(cherry picked from commit b31a1cb)
@keith keith force-pushed the ks/remove-all-bitcode-related-tests branch from a4b364d to d0d7262 Compare May 4, 2023 16:14
I imagine a new Xcode version will just remove this from the stub
keith referenced this pull request May 4, 2023
Bitcode is no longer supported for Xcode 14 and later. App Store support for uploads from Xcode prior to 14.1 will no longer be allowed after April 25.

PiperOrigin-RevId: 527298659
@keith keith changed the title Remove all bitcode related tests. Remove bitcode support May 4, 2023
@keith keith marked this pull request as ready for review May 4, 2023 19:45
@keith
Copy link
Copy Markdown
Member Author

keith commented May 4, 2023

bazelbuild/bazel#18321

Copy link
Copy Markdown
Contributor

@mattrobmattrob mattrobmattrob left a comment

Choose a reason for hiding this comment

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

What's the plan for things like @build_bazel_rules_apple//apple/internal:bitcode_support.bzl? Shouldn't all callers of something like bitcode_support.bitcode_mode_string(...) just always assume it's none now and strip bitcode, if it's present?

@keith
Copy link
Copy Markdown
Member Author

keith commented May 4, 2023

yea probably, just wanted to do the minimum to get us green again

@keith keith merged commit 5b7c929 into master May 5, 2023
@keith keith deleted the ks/remove-all-bitcode-related-tests branch May 5, 2023 15:16
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.

Rules_apple fails on Bazel@HEAD in downstream CI.

4 participants