这是indexloc提供的服务,不要输入任何密码
Skip to content

Refactor BuildInfo to always require packageConfigPath #150559

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

Merged

Conversation

sigurdm
Copy link
Contributor

@sigurdm sigurdm commented Jun 20, 2024

Refactor warming up to #150196

@github-actions github-actions bot added tool Affects the "flutter" command-line tool. See also t: labels. a: desktop Running on desktop labels Jun 20, 2024
@sigurdm sigurdm force-pushed the make_build_info_package_config_required branch from 995c8f1 to 63a249c Compare June 20, 2024 14:33
@andrewkolos andrewkolos self-requested a review June 20, 2024 23:03
@andrewkolos
Copy link
Contributor

Nit: while we are here, should we change https://github.com/flutter/flutter/pull/150559/files#diff-1f74498e0702fc7574b61a37cb0105a5c74f99d076f520756d718e341aa14532R186 from
FlutterProject.current() to super.project?

Copy link
Contributor

@andrewkolos andrewkolos left a comment

Choose a reason for hiding this comment

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

Thanks for cleaning this up! All of my comments are nits except for the one about an unused parameter.

@sigurdm sigurdm closed this Jun 25, 2024
@sigurdm sigurdm reopened this Jun 25, 2024
@sigurdm sigurdm requested a review from andrewkolos June 26, 2024 07:58
@andrewkolos
Copy link
Contributor

Google testing appears to be failing

@sigurdm
Copy link
Contributor Author

sigurdm commented Jun 27, 2024

Google testing appears to be failing

Tried investigating - I'm not sure I understand exactly what is going on. Seems a problem with library reloading.
Any help would be appreciated.

@sigurdm
Copy link
Contributor Author

sigurdm commented Jul 1, 2024

Now it seems to be passing google tests!!! (very excited - this was tough one to debug in google3 - we were not using the passed --packages when creating the the BuildInfo).

I think the other failing tests are irrelevant to this PR...

PTAL

Copy link
Contributor

@andrewkolos andrewkolos left a comment

Choose a reason for hiding this comment

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

LGTM

@sigurdm sigurdm closed this Jul 2, 2024
@sigurdm sigurdm reopened this Jul 2, 2024
@sigurdm sigurdm merged commit 21d9969 into flutter:master Jul 2, 2024
129 checks passed
sigurdm added a commit to sigurdm/flutter that referenced this pull request Jul 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 2, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jul 3, 2024
flutter/flutter@99bb2ff...af913a7

2024-07-02 andrewrkolos@gmail.com Use `ErrorHandlingFileSystem.deleteIfExists` when deleting .plugin_symlinks (flutter/flutter#151073)
2024-07-02 hans.muller@gmail.com ScrollEndNotification example: auto-scroll based on RenderSliver constraints and geometry (flutter/flutter#143538)
2024-07-02 engine-flutter-autoroll@skia.org Roll Packages from 412ec46 to d2705fb (13 revisions) (flutter/flutter#151169)
2024-07-02 goderbauer@google.com docimports for painting (flutter/flutter#151143)
2024-07-02 goderbauer@google.com docimports for scheduler (flutter/flutter#151126)
2024-07-02 nate.w5687@gmail.com `dismissible.dart` code cleanup (flutter/flutter#150276)
2024-07-02 goderbauer@google.com docimports for physics (flutter/flutter#151125)
2024-07-02 goderbauer@google.com docimports for services (flutter/flutter#151134)
2024-07-02 goderbauer@google.com docimports for cupertino (flutter/flutter#151149)
2024-07-02 goderbauer@google.com docimports for gestures (flutter/flutter#151123)
2024-07-02 goderbauer@google.com Docimports for foundation (flutter/flutter#151119)
2024-07-02 goderbauer@google.com docimports for semantics (flutter/flutter#151132)
2024-07-02 jonahwilliams@google.com [flutter_driver] add allocator mtl to memory event allowlist. (flutter/flutter#151153)
2024-07-02 engine-flutter-autoroll@skia.org Roll Flutter Engine from 40c087b31515 to 433d360eee11 (7 revisions) (flutter/flutter#151165)
2024-07-02 sigurdm@google.com Refactor BuildInfo to always require packageConfigPath (flutter/flutter#150559)
2024-07-02 engine-flutter-autoroll@skia.org Roll Flutter Engine from d3c5bd66a78f to 40c087b31515 (1 revision) (flutter/flutter#151156)
2024-07-02 engine-flutter-autoroll@skia.org Roll Flutter Engine from fc5bc14e6091 to d3c5bd66a78f (1 revision) (flutter/flutter#151155)
2024-07-02 dkwingsmt@users.noreply.github.com Fix: `CupertinoActionSheet` should take up max height when actions section is short (flutter/flutter#150708)
2024-07-02 engine-flutter-autoroll@skia.org Roll Flutter Engine from 3456fee1a6b9 to fc5bc14e6091 (8 revisions) (flutter/flutter#151150)
2024-07-02 andrewrkolos@gmail.com [tool] remove some temporary printTrace calls (flutter/flutter#151074)
2024-07-01 nate.w5687@gmail.com Implementing a few switch statements (flutter/flutter#150946)
2024-07-01 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Upgrade template Gradle, App AGP, Module AGP, and Kotlin versions, and tests (#150969)" (flutter/flutter#151147)
2024-07-01 34871572+gmackall@users.noreply.github.com Upgrade template Gradle, App AGP, Module AGP, and Kotlin versions, and tests (flutter/flutter#150969)
2024-07-01 engine-flutter-autoroll@skia.org Roll Flutter Engine from b57a044ed10f to 3456fee1a6b9 (5 revisions) (flutter/flutter#151127)
2024-07-01 codefu@google.com Read AndroidManifest.xml and emit manifest-aar-impeller-(enabled|disabled) analytics (flutter/flutter#150970)
2024-07-01 goderbauer@google.com More docimports for animation library (flutter/flutter#151011)
2024-07-01 goderbauer@google.com Bump dartdoc to 8.0.10 (flutter/flutter#151107)
2024-07-01 matthiasngeozhongsim@yahoo.com Fix missing `[` in docs (flutter/flutter#151091)
2024-07-01 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#151028)
2024-07-01 jason-simmons@users.noreply.github.com Fix teardown of a FocusScopeNode in material/app_test (flutter/flutter#151115)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC camillesimon@google.com,rmistry@google.com,stuartmorgan@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
victorsanni pushed a commit to victorsanni/flutter that referenced this pull request Jul 8, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 6, 2024
JSUYA added a commit to JSUYA/flutter-tizen that referenced this pull request Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: desktop Running on desktop tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants