Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@jmagman
Copy link
Member

@jmagman jmagman commented Sep 21, 2021

The Flutter.framework minimum version was bumped to 9.0 as of #28572 / flutter/buildroot#509. However, the framework's Info.plist was not updated, which causes App Store "Invalid bundle" submission failures. Updated it to 9.0.

The "right" fix is #28743 to set the value based on the iOS version buildroot setting. That PR also has tests, but is riskier and has a higher likelihood of being reverted.

This PR is a targeted fix to resolve the P1 ASAP.

Fixes flutter/flutter#90209

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides].
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See [testing the engine] for instructions on
    writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the [CLA].
  • All existing and new tests are passing.

@jmagman jmagman self-assigned this Sep 21, 2021
@google-cla google-cla bot added the cla: yes label Sep 21, 2021
Copy link
Member

@zanderso zanderso 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 waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Sep 22, 2021
@fluttergithubbot fluttergithubbot merged commit 4e4873e into flutter:master Sep 22, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2021
@jmagman jmagman deleted the ios-9 branch September 22, 2021 17:34
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2021
GaryQian pushed a commit to flutter/flutter that referenced this pull request Sep 22, 2021
* ffdaa4b Implement a default font manager for Fuchsia. (flutter/engine#28673)

* 0608a01 Replace jcenter() for mavenCentral() (flutter/engine#28777)

* ccec08a Roll Dart SDK from a0243abd09e3 to 86a21d182523 (6 revisions) (flutter/engine#28779)

* e2bd498 [web] clear surfaces on hot restart (flutter/engine#28675)

* 6eceff5 Roll Skia from e32309d771ee to a48e7b0186b3 (10 revisions) (flutter/engine#28782)

* 3c01c60 Eliminate unnecessary canvaskit imports (flutter/engine#28787)

* 4e4873e Set MinimumOSVersion to iOS 9 in Info.plist (flutter/engine#28783)

* 4872f8f Roll Dart SDK from 86a21d182523 to c2429dee845e (1 revision) (flutter/engine#28789)

* d8120eb Roll Skia from a48e7b0186b3 to bb30fc16e1be (1 revision) (flutter/engine#28791)

* 13a631a Roll Fuchsia Mac SDK from 3k3-vCrm5... to NM9FaaYay... (flutter/engine#28792)

* a60c0c3 Roll Fuchsia Linux SDK from JD40LpEJO... to bWZcp2jnx... (flutter/engine#28793)

* 04d47cd Roll Skia from bb30fc16e1be to 5527735121f5 (1 revision) (flutter/engine#28794)

* 827c62a Roll Dart SDK from c2429dee845e to 715a98df1828 (1 revision) (flutter/engine#28795)

* f16f0ce Roll Skia from 5527735121f5 to cb25d566c21d (3 revisions) (flutter/engine#28796)

* ec009a3 Roll Dart SDK from 715a98df1828 to 4e396689de4f (1 revision) (flutter/engine#28797)

* df5b566 Disable failing unit tests on UWP (flutter/engine#28785)

* b6e7440 Roll Dart SDK from 4e396689de4f to 6e17953f4e7e (1 revision) (flutter/engine#28800)

* fa2eb22 [canvaskit] Fix bug when overlays are disabled (flutter/engine#28790)
xster pushed a commit to xster/engine that referenced this pull request Sep 30, 2021
xster added a commit that referenced this pull request Oct 4, 2021
Ignoring the web framework test since the failure originates from a repo synchronization issue between engine and framework. The new failing tests were sync'ed in from head in the test and the tested code doesn't exist in the engine at the time of branch cut. 

* Fixes FlutterSemanticsScrollView to not implement accessibility container API (#28846)

* add branch to test

* Replace jcenter() for mavenCentral() (#28777)

* Set MinimumOSVersion to iOS 9 in Info.plist (#28783)

* Do not use jcenter(), switch to mavenCentral() (#28738)

Co-authored-by: chunhtai <[email protected]>
Co-authored-by: Emmanuel Garcia <[email protected]>
Co-authored-by: Jenn Magder <[email protected]>
Co-authored-by: Dan Field <[email protected]>
clocksmith pushed a commit to clocksmith/flutter that referenced this pull request Oct 29, 2021
* ffdaa4b Implement a default font manager for Fuchsia. (flutter/engine#28673)

* 0608a01 Replace jcenter() for mavenCentral() (flutter/engine#28777)

* ccec08a Roll Dart SDK from a0243abd09e3 to 86a21d182523 (6 revisions) (flutter/engine#28779)

* e2bd498 [web] clear surfaces on hot restart (flutter/engine#28675)

* 6eceff5 Roll Skia from e32309d771ee to a48e7b0186b3 (10 revisions) (flutter/engine#28782)

* 3c01c60 Eliminate unnecessary canvaskit imports (flutter/engine#28787)

* 4e4873e Set MinimumOSVersion to iOS 9 in Info.plist (flutter/engine#28783)

* 4872f8f Roll Dart SDK from 86a21d182523 to c2429dee845e (1 revision) (flutter/engine#28789)

* d8120eb Roll Skia from a48e7b0186b3 to bb30fc16e1be (1 revision) (flutter/engine#28791)

* 13a631a Roll Fuchsia Mac SDK from 3k3-vCrm5... to NM9FaaYay... (flutter/engine#28792)

* a60c0c3 Roll Fuchsia Linux SDK from JD40LpEJO... to bWZcp2jnx... (flutter/engine#28793)

* 04d47cd Roll Skia from bb30fc16e1be to 5527735121f5 (1 revision) (flutter/engine#28794)

* 827c62a Roll Dart SDK from c2429dee845e to 715a98df1828 (1 revision) (flutter/engine#28795)

* f16f0ce Roll Skia from 5527735121f5 to cb25d566c21d (3 revisions) (flutter/engine#28796)

* ec009a3 Roll Dart SDK from 715a98df1828 to 4e396689de4f (1 revision) (flutter/engine#28797)

* df5b566 Disable failing unit tests on UWP (flutter/engine#28785)

* b6e7440 Roll Dart SDK from 4e396689de4f to 6e17953f4e7e (1 revision) (flutter/engine#28800)

* fa2eb22 [canvaskit] Fix bug when overlays are disabled (flutter/engine#28790)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes platform-ios waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

3 participants