Skip to content

Remove generated file from template manifest - #177034

Merged
auto-submit[bot] merged 3 commits into
flutter:masterfrom
robert-ancell:linux-template-manifest
Oct 29, 2025
Merged

auto-submit[bot] merged 3 commits into
flutter:masterfrom
robert-ancell:linux-template-manifest

Conversation

@robert-ancell

Copy link
Copy Markdown
Contributor

I'm not sure what this manifest is being used for but none of the other templates also list the file that is generated from them, seems to have been a typo?

@github-actions github-actions Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Oct 15, 2025

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request correctly removes an entry for a generated file, templates/app/linux.tmpl/runner/my_application.cc, from template_manifest.json. As you suspected, this file is generated from my_application.cc.tmpl and does not belong in the manifest of template source files. This change makes the manifest more accurate.

@robert-ancell

Copy link
Copy Markdown
Contributor Author

@loic-sharma adding you as a reviewer in case you know the right person to check this.

@loic-sharma
loic-sharma requested a review from bkonyi October 24, 2025 15:35
@loic-sharma

loic-sharma commented Oct 24, 2025 •

Copy link
Copy Markdown
Member

@bkonyi as the flutter tool tech lead would be the right reviewer, but this LGTM :)

Some interesting background reading for templates manifests:

  1. Create a manifest for template (.tmpl) files and use it as the source of truth #59602
  2. flutter create failed - "Value was missing for variable tag: androidFlutterJar" #57985 (comment)

The template manifest seems to be an allowlist of files that can be used from the templates, in case the templates incorrectly have file.

@bkonyi bkonyi added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 24, 2025
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 24, 2025
@auto-submit

auto-submit Bot commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/flutter/177034, because The base commit of the PR is older than 7 days and can not be merged. Please merge the latest changes from the main into this branch and resubmit the PR.

@bkonyi bkonyi added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 24, 2025
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 24, 2025
@auto-submit

auto-submit Bot commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/flutter/177034, because - The status or check suite ci.yaml validation has failed. Please fix the issues identified (or deflake) before re-applying this label.

@robert-ancell robert-ancell added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 29, 2025
@auto-submit
auto-submit Bot added this pull request to the merge queue Oct 29, 2025
Merged via the queue into flutter:master with commit 3f4fcd7 Oct 29, 2025
148 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 29, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 29, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 29, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 29, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 29, 2025
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Oct 29, 2025
flutter/flutter@7cf0dc1...df72035

2025-10-29 [email protected] Roll Packages from c8ba0cc to 41c6b3d (3 revisions) (flutter/flutter#177725)
2025-10-29 [email protected] [Android] Remove unecessary `spy` in `FlutterActivityAndFragmentDelegateTest` (flutter/flutter#177120)
2025-10-29 [email protected] Add discussion of federated plugin documentation (flutter/flutter#177659)
2025-10-29 [email protected] [web] Deprecate --pwa-strategy (flutter/flutter#177613)
2025-10-29 [email protected] Roll Skia from 53b8b802bbc4 to 0a0c9f8c704f (1 revision) (flutter/flutter#177701)
2025-10-29 [email protected] Add `Navigator.popUntilWithResult` (flutter/flutter#169341)
2025-10-29 [email protected] Replace deprecated `withOpacity` with  `withValues` in `text_style.dart` (flutter/flutter#177537)
2025-10-29 [email protected] Replace deprecated withOpacity in `radio.1.dart` example (flutter/flutter#177606)
2025-10-29 [email protected] Roll Skia from 4408d8ea88b0 to 53b8b802bbc4 (2 revisions) (flutter/flutter#177698)
2025-10-29 [email protected] Remove generated file from template manifest (flutter/flutter#177034)
2025-10-29 [email protected] Roll Skia from e582a5594b6f to 4408d8ea88b0 (5 revisions) (flutter/flutter#177691)
2025-10-28 [email protected] Roll Fuchsia Linux SDK from ir6J2isKAYa1jNLyJ... to 3EF6k6lqXPWDwrdyj... (flutter/flutter#177682)
2025-10-28 [email protected] [Gradle 9] Fix Engine Deps (flutter/flutter#177623)
2025-10-28 [email protected] Replace deprecated `withOpacity` in `focus_scope.0.dart‎` example  (flutter/flutter#177542)
2025-10-28 [email protected] Fix EditableText _justResumed is not accurate (flutter/flutter#177658)
2025-10-28 [email protected] Replace deprecated `withOpacity` in `interactive_viewer.builder.0.dart` (flutter/flutter#177541)
2025-10-28 [email protected] Fix TextButton.icon breaks focus traversal and ink effect when toggling icon (flutter/flutter#176579)
2025-10-28 [email protected] [Android 16] Update Engine `ci.yaml` to test against Java 21 (flutter/flutter#177677)
2025-10-28 [email protected] Replace deprecated `withOpacity` in `interactive_viewer.constrained.0.dart` (flutter/flutter#177540)
2025-10-28 [email protected] Replace opacity from random color in navigation bar test (flutter/flutter#177490)
2025-10-28 [email protected] Roll Skia from e4d3d8f31aef to e582a5594b6f (6 revisions) (flutter/flutter#177679)
2025-10-28 [email protected] Workaround for lag when dragging window titlebar on Windows (flutter/flutter#177597)
2025-10-28 [email protected] Roll Packages from bbf96a0 to c8ba0cc (2 revisions) (flutter/flutter#177672)

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://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
walley892 pushed a commit to walley892/flutter that referenced this pull request Oct 30, 2025
I'm not sure what this manifest is being used for but none of the other
templates also list the file that is generated from them, seems to have
been a typo?

Co-authored-by: Ben Konyi <[email protected]>
reidbaker pushed a commit to AbdeMohlbi/flutter that referenced this pull request Dec 10, 2025
I'm not sure what this manifest is being used for but none of the other
templates also list the file that is generated from them, seems to have
been a typo?

Co-authored-by: Ben Konyi <[email protected]>
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 11, 2026
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
flutter/flutter@7cf0dc1...df72035

2025-10-29 [email protected] Roll Packages from c8ba0cc to 41c6b3d (3 revisions) (flutter/flutter#177725)
2025-10-29 [email protected] [Android] Remove unecessary `spy` in `FlutterActivityAndFragmentDelegateTest` (flutter/flutter#177120)
2025-10-29 [email protected] Add discussion of federated plugin documentation (flutter/flutter#177659)
2025-10-29 [email protected] [web] Deprecate --pwa-strategy (flutter/flutter#177613)
2025-10-29 [email protected] Roll Skia from 53b8b802bbc4 to 0a0c9f8c704f (1 revision) (flutter/flutter#177701)
2025-10-29 [email protected] Add `Navigator.popUntilWithResult` (flutter/flutter#169341)
2025-10-29 [email protected] Replace deprecated `withOpacity` with  `withValues` in `text_style.dart` (flutter/flutter#177537)
2025-10-29 [email protected] Replace deprecated withOpacity in `radio.1.dart` example (flutter/flutter#177606)
2025-10-29 [email protected] Roll Skia from 4408d8ea88b0 to 53b8b802bbc4 (2 revisions) (flutter/flutter#177698)
2025-10-29 [email protected] Remove generated file from template manifest (flutter/flutter#177034)
2025-10-29 [email protected] Roll Skia from e582a5594b6f to 4408d8ea88b0 (5 revisions) (flutter/flutter#177691)
2025-10-28 [email protected] Roll Fuchsia Linux SDK from ir6J2isKAYa1jNLyJ... to 3EF6k6lqXPWDwrdyj... (flutter/flutter#177682)
2025-10-28 [email protected] [Gradle 9] Fix Engine Deps (flutter/flutter#177623)
2025-10-28 [email protected] Replace deprecated `withOpacity` in `focus_scope.0.dart‎` example  (flutter/flutter#177542)
2025-10-28 [email protected] Fix EditableText _justResumed is not accurate (flutter/flutter#177658)
2025-10-28 [email protected] Replace deprecated `withOpacity` in `interactive_viewer.builder.0.dart` (flutter/flutter#177541)
2025-10-28 [email protected] Fix TextButton.icon breaks focus traversal and ink effect when toggling icon (flutter/flutter#176579)
2025-10-28 [email protected] [Android 16] Update Engine `ci.yaml` to test against Java 21 (flutter/flutter#177677)
2025-10-28 [email protected] Replace deprecated `withOpacity` in `interactive_viewer.constrained.0.dart` (flutter/flutter#177540)
2025-10-28 [email protected] Replace opacity from random color in navigation bar test (flutter/flutter#177490)
2025-10-28 [email protected] Roll Skia from e4d3d8f31aef to e582a5594b6f (6 revisions) (flutter/flutter#177679)
2025-10-28 [email protected] Workaround for lag when dragging window titlebar on Windows (flutter/flutter#177597)
2025-10-28 [email protected] Roll Packages from bbf96a0 to c8ba0cc (2 revisions) (flutter/flutter#177672)

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://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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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