Skip to content

Conversation

@andrewkolos
Copy link
Contributor

Closes #128456, which is now linked to in a code comment in this change.
Reopens #124883.

This effectively reverts #126077 and is intended to be cherry-picked into stable.

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, including Features we expect every widget to implement.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@andrewkolos andrewkolos added tool Affects the "flutter" command-line tool. See also t: labels. a: assets Packaging, accessing, or using assets labels Jun 8, 2023
@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Jun 8, 2023
@andrewkolos andrewkolos requested a review from eliasyishak June 8, 2023 19:56
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@andrewkolos
Copy link
Contributor Author

I updated this PR with a stopgap fix to prevent #127297 from reoccurring once this users upgrade to a patch that includes this fix. See this comment for context and this commit for the change.

Copy link
Contributor

@eliasyishak eliasyishak left a comment

Choose a reason for hiding this comment

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

lgtm! :) sorry for the delay

@andrewkolos andrewkolos added the autosubmit Merge PR when tree becomes green via auto submit App label Jun 9, 2023
@auto-submit auto-submit bot merged commit cfe4fed into flutter:master Jun 9, 2023
andrewkolos added a commit to andrewkolos/flutter that referenced this pull request Jun 9, 2023
…m `AssetManifest.smcbin`) (flutter#128529)

Closes flutter#128456, which is now linked to in a code comment in this change.
Reopens flutter#124883.

This effectively reverts flutter#126077 and is intended to be cherry-picked into stable.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 11, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 11, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jun 11, 2023
flutter/flutter@da127f1...3df163f

2023-06-11 [email protected] Roll Flutter Engine from 73a7af805472 to 1cca9cc6dbd1 (1 revision) (flutter/flutter#128658)
2023-06-11 [email protected] Roll Flutter Engine from 7c6770083e5c to 73a7af805472 (2 revisions) (flutter/flutter#128654)
2023-06-11 [email protected] Roll Flutter Engine from 6d615bbcfccf to 7c6770083e5c (2 revisions) (flutter/flutter#128653)
2023-06-11 [email protected] Roll Flutter Engine from b19b93de5b0a to 6d615bbcfccf (1 revision) (flutter/flutter#128650)
2023-06-11 [email protected] Roll Flutter Engine from 3d76ba6d6d5f to b19b93de5b0a (2 revisions) (flutter/flutter#128649)
2023-06-11 [email protected] Roll Flutter Engine from 962d78e0ae9c to 3d76ba6d6d5f (1 revision) (flutter/flutter#128645)
2023-06-10 [email protected] migrate `Tooltip` to use  `OverlayPortal` (flutter/flutter#127728)
2023-06-10 [email protected] Roll Flutter Engine from b037db26037f to 962d78e0ae9c (10 revisions) (flutter/flutter#128643)
2023-06-10 49699333+dependabot[bot]@users.noreply.github.com Bump actions/checkout from 3.5.2 to 3.5.3 (flutter/flutter#128625)
2023-06-10 [email protected] Roll Flutter Engine from 3e90345cdca7 to b037db26037f (1 revision) (flutter/flutter#128627)
2023-06-10 [email protected] Remove unnecessary parentheses (flutter/flutter#128620)
2023-06-09 [email protected] Roll Flutter Engine from 488876ed26c6 to 3e90345cdca7 (3 revisions) (flutter/flutter#128617)
2023-06-09 [email protected] rename generated asset manifest file back to `AssetManifest.bin` (from `AssetManifest.smcbin`) (flutter/flutter#128529)
2023-06-09 [email protected] Add Selected semantics to IconButton (flutter/flutter#128547)
2023-06-09 [email protected] Roll Flutter Engine from 071e1fb21c7a to 488876ed26c6 (5 revisions) (flutter/flutter#128612)
2023-06-09 [email protected] Clarifies semantics long press and semantics on tap documentation (flutter/flutter#128599)
2023-06-09 [email protected] Roll Flutter Engine from bc6e047570f6 to 071e1fb21c7a (1 revision) (flutter/flutter#128602)
2023-06-09 [email protected] Revert "Update `chip.dart` to use set of `MaterialState`" (flutter/flutter#128607)
2023-06-09 [email protected] Add tooltips for `SegmentedButton` (flutter/flutter#128501)
2023-06-09 [email protected] Ignore app.stop events received before the app.detach response in attach integration tests (flutter/flutter#128593)
2023-06-09 [email protected] ExpansionPanel isExpanded callback parameter (Ticket 74114) (flutter/flutter#128082)
2023-06-09 [email protected] Roll Flutter Engine from 93afba901b3b to bc6e047570f6 (3 revisions) (flutter/flutter#128594)
2023-06-09 [email protected] Updated flutter_localizations tests for Material3; (flutter/flutter#128521)
2023-06-09 [email protected] Paint SelectableFragments before text (flutter/flutter#128375)
2023-06-09 [email protected] Roll Packages from e13b8c4 to afe2f05 (7 revisions) (flutter/flutter#128582)

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 [email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
auto-submit bot pushed a commit that referenced this pull request Jun 12, 2023
… (from `AssetManifest.smcbin`) (#128621)

Cherry-picks #128529 onto stable.
Fixes #128230
@andrewkolos andrewkolos deleted the change-asset-manifest-extension-back-to-bin branch June 15, 2023 00:57
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 16, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: assets Packaging, accessing, or using assets autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: 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.

Using uncommon file extensions for flutter-internal files risks customer web servers refusing to serve them

3 participants