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

Add bitcode deprecation note for add-to-app iOS developers #112900

Merged
merged 3 commits into from
Oct 6, 2022

Conversation

jmagman
Copy link
Member

@jmagman jmagman commented Oct 5, 2022

Bitcode has been deprecated. Existing Flutter apps were migrated in #112828, but we have no control over iOS add-to-app host apps. Give developers a warning that they need to turn off the build setting in the Xcode host app.

Screen Shot 2022-10-04 at 6 04 39 PM

@jmagman jmagman added platform-ios iOS applications specifically tool Affects the "flutter" command-line tool. See also t: labels. labels Oct 5, 2022
@jmagman jmagman self-assigned this Oct 5, 2022
@flutter-dashboard flutter-dashboard bot added the team Infra upgrades, team productivity, code health, technical debt. See also team: labels. label Oct 5, 2022
@jmagman jmagman mentioned this pull request Oct 5, 2022
17 tasks
@@ -87,7 +87,8 @@ Future<void> _testBuildIosFramework(Directory projectDir, { bool isModule = fals
const String outputDirectoryName = 'flutter-frameworks';

await inDirectory(projectDir, () async {
await flutter(
final StringBuffer outputError = StringBuffer();
final String output = await evalFlutter(
Copy link
Member

Choose a reason for hiding this comment

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

RUNNING: cd .; bin/flutter analyze --flutter-repo
Analyzing 3 items...                                            

   info • The value of the local variable 'output' isn't used • dev/devicelab/bin/tasks/build_ios_framework_module_test.dart:91:18 • unused_local_variable

1 issue found. (ran in 122.4s)

Copy link
Member

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM

@jmagman jmagman added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 6, 2022
@auto-submit auto-submit bot merged commit 7dd65c1 into flutter:master Oct 6, 2022
@jmagman jmagman deleted the bitcode-build-frameworks branch October 6, 2022 03:43
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Oct 6, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Oct 6, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Oct 6, 2022
auto-submit bot pushed a commit that referenced this pull request May 20, 2024
Flutter deprecated bitcode in 2022 #112828, and introduced a warning in #112900 to let add-to-app devs know.

Apps should be migrated by now, remove the outdated warning.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-ios iOS applications specifically team Infra upgrades, team productivity, code health, technical debt. See also team: labels. tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants