Skip to content

Space character should be optional when tree shaking fonts - #132880

Merged
auto-submit[bot] merged 1 commit into
flutter:masterfrom
eyebrowsoffire:icon_tree_shake_optional_space
Aug 19, 2023
Merged

auto-submit[bot] merged 1 commit into
flutter:masterfrom
eyebrowsoffire:icon_tree_shake_optional_space

Conversation

@eyebrowsoffire

@eyebrowsoffire eyebrowsoffire commented Aug 19, 2023 •

Copy link
Copy Markdown
Contributor

Addresses the other part of #132711

@github-actions github-actions Bot added tool Affects the "flutter" command-line tool. See also t: labels. engine flutter/engine related. See also e: labels. labels Aug 19, 2023
@eyebrowsoffire
eyebrowsoffire force-pushed the icon_tree_shake_optional_space branch from 4ae1b58 to 94ead9e Compare August 19, 2023 00:51
@eyebrowsoffire
eyebrowsoffire marked this pull request as ready for review August 19, 2023 00:51
@eyebrowsoffire eyebrowsoffire added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 19, 2023
@eyebrowsoffire

Copy link
Copy Markdown
Contributor Author

I tested this against the repository provided by the originator of the bug and confirmed that it fixes the issue, and the application builds.

@auto-submit
auto-submit Bot merged commit 6f227c0 into flutter:master Aug 19, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 19, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 19, 2023
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Aug 19, 2023
flutter/flutter@5a556f8...6f227c0

2023-08-19 [email protected] Space character should be optional when tree shaking fonts (flutter/flutter#132880)
2023-08-19 [email protected] Roll Flutter Engine from 654d3a2c8494 to f4bffdcf8536 (5 revisions) (flutter/flutter#132876)
2023-08-18 [email protected] SelectionArea on iOS should toggle the context menu when tapping on the previous selection (flutter/flutter#132851)
2023-08-18 [email protected] Forward port API docs creation realm handling from prior script (flutter/flutter#132867)
2023-08-18 [email protected] Roll Flutter Engine from 72c4e61fdbfa to 654d3a2c8494 (3 revisions) (flutter/flutter#132868)
2023-08-18 [email protected] Roll Flutter Engine from 83fdadaf12f0 to 72c4e61fdbfa (2 revisions) (flutter/flutter#132861)
2023-08-18 [email protected] Updates app link gradle tasks and remove vm services (flutter/flutter#131805)
2023-08-18 [email protected] Roll Flutter Engine from 58f7d8ee3e2c to 83fdadaf12f0 (7 revisions) (flutter/flutter#132857)
2023-08-18 [email protected] doc: add flag params (flutter/flutter#132485)
2023-08-18 [email protected] Add a new MatrixTransition and refactor ScaleTransition and RotationTâ�¦ (flutter/flutter#131084)
2023-08-18 [email protected] fixes l10n for CupertinoDatePicker in monthYear mode (flutter/flutter#130934)

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],[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
christopherfujino pushed a commit to chris-forks/flutter that referenced this pull request Aug 21, 2023
zanderso pushed a commit to eyebrowsoffire/flutter that referenced this pull request Aug 23, 2023
auto-submit Bot pushed a commit that referenced this pull request Aug 23, 2023
@mhbdev

mhbdev commented Oct 29, 2023

Copy link
Copy Markdown
Contributor

Is the problem solved in Flutter stable 3.13.9? Cause I am facing :

Target web_release_bundle failed: IconTreeShakerException: Font subsetting failed with exit code -1.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 15, 2023
@eyebrowsoffire
eyebrowsoffire deleted the icon_tree_shake_optional_space branch December 12, 2025 21:56
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
flutter/flutter@5a556f8...6f227c0

2023-08-19 [email protected] Space character should be optional when tree shaking fonts (flutter/flutter#132880)
2023-08-19 [email protected] Roll Flutter Engine from 654d3a2c8494 to f4bffdcf8536 (5 revisions) (flutter/flutter#132876)
2023-08-18 [email protected] SelectionArea on iOS should toggle the context menu when tapping on the previous selection (flutter/flutter#132851)
2023-08-18 [email protected] Forward port API docs creation realm handling from prior script (flutter/flutter#132867)
2023-08-18 [email protected] Roll Flutter Engine from 72c4e61fdbfa to 654d3a2c8494 (3 revisions) (flutter/flutter#132868)
2023-08-18 [email protected] Roll Flutter Engine from 83fdadaf12f0 to 72c4e61fdbfa (2 revisions) (flutter/flutter#132861)
2023-08-18 [email protected] Updates app link gradle tasks and remove vm services (flutter/flutter#131805)
2023-08-18 [email protected] Roll Flutter Engine from 58f7d8ee3e2c to 83fdadaf12f0 (7 revisions) (flutter/flutter#132857)
2023-08-18 [email protected] doc: add flag params (flutter/flutter#132485)
2023-08-18 [email protected] Add a new MatrixTransition and refactor ScaleTransition and RotationTâ�¦ (flutter/flutter#131084)
2023-08-18 [email protected] fixes l10n for CupertinoDatePicker in monthYear mode (flutter/flutter#130934)

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],[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
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
flutter/flutter@5a556f8...6f227c0

2023-08-19 [email protected] Space character should be optional when tree shaking fonts (flutter/flutter#132880)
2023-08-19 [email protected] Roll Flutter Engine from 654d3a2c8494 to f4bffdcf8536 (5 revisions) (flutter/flutter#132876)
2023-08-18 [email protected] SelectionArea on iOS should toggle the context menu when tapping on the previous selection (flutter/flutter#132851)
2023-08-18 [email protected] Forward port API docs creation realm handling from prior script (flutter/flutter#132867)
2023-08-18 [email protected] Roll Flutter Engine from 72c4e61fdbfa to 654d3a2c8494 (3 revisions) (flutter/flutter#132868)
2023-08-18 [email protected] Roll Flutter Engine from 83fdadaf12f0 to 72c4e61fdbfa (2 revisions) (flutter/flutter#132861)
2023-08-18 [email protected] Updates app link gradle tasks and remove vm services (flutter/flutter#131805)
2023-08-18 [email protected] Roll Flutter Engine from 58f7d8ee3e2c to 83fdadaf12f0 (7 revisions) (flutter/flutter#132857)
2023-08-18 [email protected] doc: add flag params (flutter/flutter#132485)
2023-08-18 [email protected] Add a new MatrixTransition and refactor ScaleTransition and RotationTâ�¦ (flutter/flutter#131084)
2023-08-18 [email protected] fixes l10n for CupertinoDatePicker in monthYear mode (flutter/flutter#130934)

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],[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
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 engine flutter/engine related. See also e: 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.

3 participants