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

Remove bitcode support #1534

Merged
merged 4 commits into from
May 5, 2023
Merged

Remove bitcode support #1534

merged 4 commits into from
May 5, 2023

Conversation

keith
Copy link
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
Member Author

keith commented May 4, 2023

bazelbuild/bazel#18321

Copy link
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
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
mattrobmattrob added a commit to mattrobmattrob/rules_ios that referenced this pull request Jun 1, 2023
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