Skip to content

Remove use of deprecated copy_trees. - #187091

Merged
auto-submit[bot] merged 1 commit into
flutter:masterfrom
rmacnak-google:remove-deprecated-copy-trees
May 26, 2026
Merged

auto-submit[bot] merged 1 commit into
flutter:masterfrom
rmacnak-google:remove-deprecated-copy-trees

Conversation

@rmacnak-google

Copy link
Copy Markdown
Contributor

No description provided.

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 26, 2026
@github-actions github-actions Bot added the engine flutter/engine related. See also e: labels. label May 26, 2026
@rmacnak-google
rmacnak-google force-pushed the remove-deprecated-copy-trees branch from 7b74f06 to e6ce10d Compare May 26, 2026 16:24
@github-actions github-actions Bot removed the CICD Run CI/CD label May 26, 2026
@rmacnak-google rmacnak-google added the CICD Run CI/CD label May 26, 2026
@rmacnak-google
rmacnak-google marked this pull request as ready for review May 26, 2026 17:55

@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 simplifies GN build targets by replacing the copy_trees template with copy_tree in both the Dart SDK and iOS scenario app build configurations, updating corresponding CI builder JSON files to reference the new target names. The review feedback highlights potential regressions where renaming or removing existing targets (scenario_ios and copy_dart_sdk) and restricting visibility could break external dependencies or other builder configurations. It is recommended to maintain compatibility targets to prevent build failures.

Comment thread engine/src/flutter/testing/ios_scenario_app/ios/BUILD.gn
Comment thread engine/src/flutter/build/dart/BUILD.gn
@rmacnak-google rmacnak-google added the autosubmit Merge PR when tree becomes green via auto submit App label May 26, 2026
@auto-submit
auto-submit Bot added this pull request to the merge queue May 26, 2026
Merged via the queue into flutter:master with commit 6941cd3 May 26, 2026
206 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label May 26, 2026
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request May 27, 2026
flutter/flutter@f3a4b98...c8f2f16

2026-05-27 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Flutter GPU] Add explicit draw counts (#186639)" (flutter/flutter#187170)
2026-05-27 [email protected] Roll Dart SDK from 7dcea971af6b to f3db7b7d9801 (2 revisions) (flutter/flutter#187144)
2026-05-27 [email protected] [Flutter GPU] Add explicit draw counts (flutter/flutter#186639)
2026-05-27 [email protected] Roll Skia from f9db7748563e to fa944af10f91 (1 revision) (flutter/flutter#187139)
2026-05-27 [email protected] [Flutter GPU] Flutter GPU ShaderLibrary in-place reload for shader hot reload (flutter/flutter#186793)
2026-05-27 [email protected] Roll Skia from a692cbf38952 to f9db7748563e (8 revisions) (flutter/flutter#187135)
2026-05-27 [email protected] Use local semantics tester in Material button tests (flutter/flutter#186667)
2026-05-27 [email protected] Filter out `a: text input` from `team-framework` PR triage  (flutter/flutter#187129)
2026-05-27 [email protected] [Engine] Fix silent buffer mismatch bug in FML Win32 CommandLineFromPlatform (flutter/flutter#187120)
2026-05-27 [email protected] examples: Remove unused parameters (flutter/flutter#185819)
2026-05-27 [email protected] Remove another `String.valueOf` (flutter/flutter#186628)
2026-05-27 [email protected] Print trace when skipping flavor-specific and platform-specific assets (flutter/flutter#187045)
2026-05-26 [email protected] Fix `InteractiveViewer` memory leak from undisposed `CurvedAnimation`s (flutter/flutter#185826)
2026-05-26 [email protected] Fix AccessibilityBridge startup crash on vendor-modified Android (flutter/flutter#184630)
2026-05-26 [email protected] Update data driven fixes docs (flutter/flutter#186217)
2026-05-26 [email protected] Add more 0x0 size tests part10 (flutter/flutter#186201)
2026-05-26 [email protected] Disable spell check for obscured text (flutter/flutter#186329)
2026-05-26 [email protected] [iOS] Migrate VSyncClient and DisplayLinkManager to Swift (flutter/flutter#187001)
2026-05-26 [email protected] [flutter_tools, devicelab] Add safety filesystem guard to tests (flutter/flutter#186748)
2026-05-26 [email protected] Roll Fuchsia Linux SDK from Itd2Jq_ZIABH2rW7B... to k9EizfEGJO4WwQN9-... (flutter/flutter#187115)
2026-05-26 [email protected] Roll Dart SDK from 00e625453c43 to 7dcea971af6b (1 revision) (flutter/flutter#187117)
2026-05-26 [email protected] Remove use of deprecated copy_trees. (flutter/flutter#187091)
2026-05-26 [email protected] Use resolved implementation plugins with SwiftPM (flutter/flutter#187097)
2026-05-26 [email protected] Roll Skia from 27a819894f7c to a692cbf38952 (3 revisions) (flutter/flutter#187109)
2026-05-26 [email protected] Roll Packages from 69cf959 to fc795e5 (4 revisions) (flutter/flutter#187114)
2026-05-26 [email protected] Handle simctl process exceptions during discovery (flutter/flutter#186501)
2026-05-26 [email protected] Clarify precache enabled platforms help (flutter/flutter#186878)
2026-05-26 [email protected] [tool] Refactor artifacts.dart to use enhanced enums and reduce duplication (flutter/flutter#187063)
2026-05-26 [email protected] Build script updates for syncing engine sources and building artifacts on linux-arm64 (flutter/flutter#186917)

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
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…r#11795)

flutter/flutter@f3a4b98...c8f2f16

2026-05-27 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Flutter GPU] Add explicit draw counts (#186639)" (flutter/flutter#187170)
2026-05-27 [email protected] Roll Dart SDK from 7dcea971af6b to f3db7b7d9801 (2 revisions) (flutter/flutter#187144)
2026-05-27 [email protected] [Flutter GPU] Add explicit draw counts (flutter/flutter#186639)
2026-05-27 [email protected] Roll Skia from f9db7748563e to fa944af10f91 (1 revision) (flutter/flutter#187139)
2026-05-27 [email protected] [Flutter GPU] Flutter GPU ShaderLibrary in-place reload for shader hot reload (flutter/flutter#186793)
2026-05-27 [email protected] Roll Skia from a692cbf38952 to f9db7748563e (8 revisions) (flutter/flutter#187135)
2026-05-27 [email protected] Use local semantics tester in Material button tests (flutter/flutter#186667)
2026-05-27 [email protected] Filter out `a: text input` from `team-framework` PR triage  (flutter/flutter#187129)
2026-05-27 [email protected] [Engine] Fix silent buffer mismatch bug in FML Win32 CommandLineFromPlatform (flutter/flutter#187120)
2026-05-27 [email protected] examples: Remove unused parameters (flutter/flutter#185819)
2026-05-27 [email protected] Remove another `String.valueOf` (flutter/flutter#186628)
2026-05-27 [email protected] Print trace when skipping flavor-specific and platform-specific assets (flutter/flutter#187045)
2026-05-26 [email protected] Fix `InteractiveViewer` memory leak from undisposed `CurvedAnimation`s (flutter/flutter#185826)
2026-05-26 [email protected] Fix AccessibilityBridge startup crash on vendor-modified Android (flutter/flutter#184630)
2026-05-26 [email protected] Update data driven fixes docs (flutter/flutter#186217)
2026-05-26 [email protected] Add more 0x0 size tests part10 (flutter/flutter#186201)
2026-05-26 [email protected] Disable spell check for obscured text (flutter/flutter#186329)
2026-05-26 [email protected] [iOS] Migrate VSyncClient and DisplayLinkManager to Swift (flutter/flutter#187001)
2026-05-26 [email protected] [flutter_tools, devicelab] Add safety filesystem guard to tests (flutter/flutter#186748)
2026-05-26 [email protected] Roll Fuchsia Linux SDK from Itd2Jq_ZIABH2rW7B... to k9EizfEGJO4WwQN9-... (flutter/flutter#187115)
2026-05-26 [email protected] Roll Dart SDK from 00e625453c43 to 7dcea971af6b (1 revision) (flutter/flutter#187117)
2026-05-26 [email protected] Remove use of deprecated copy_trees. (flutter/flutter#187091)
2026-05-26 [email protected] Use resolved implementation plugins with SwiftPM (flutter/flutter#187097)
2026-05-26 [email protected] Roll Skia from 27a819894f7c to a692cbf38952 (3 revisions) (flutter/flutter#187109)
2026-05-26 [email protected] Roll Packages from 69cf959 to fc795e5 (4 revisions) (flutter/flutter#187114)
2026-05-26 [email protected] Handle simctl process exceptions during discovery (flutter/flutter#186501)
2026-05-26 [email protected] Clarify precache enabled platforms help (flutter/flutter#186878)
2026-05-26 [email protected] [tool] Refactor artifacts.dart to use enhanced enums and reduce duplication (flutter/flutter#187063)
2026-05-26 [email protected] Build script updates for syncing engine sources and building artifacts on linux-arm64 (flutter/flutter#186917)

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
camsim99 pushed a commit to flutteractionsbot/flutter that referenced this pull request Jun 10, 2026
@camsim99 camsim99 added the cp: beta cherry pick this pull request to beta release candidate branch label Jun 11, 2026
camsim99 pushed a commit to camsim99/flutter that referenced this pull request Jun 11, 2026
auto-submit Bot pushed a commit that referenced this pull request Jun 12, 2026
….pre release (#187872)

Updates DEPS to update the Dart revision to aad8be4ce3077a809f0b4f3325c6d8a6908aad54 and makes the following changes that Dart version update (this list includes all of the commits in this PR!!):

| Commit  | Reasoning |
| ------------- | ------------- |
|9cddb59 | New Dart version removed `ai` repository from their internal DEPS in dart-lang/sdk@06bdbd7 (cherry pick of #187378)  |
| a6fb7ec | Actual commit that updates Dart version 🎯  |
| 527ebc7 | Required to compile the engine with the new Dart version (cherry pick of #187013) |
| 37c44d7 | Also required to compile the engine with the new Dart version (cherry pick of #187091) |
| aafb985 | Ignore; reverted in 03b6bf1 |
| dc91998 | Update LICENSES according to new Dart version |
| 2cd036e | Ignore; reverted in 03b6bf1 |
| 50a3ace | Formats engine Dart files according to updated formatter in new version of Dart SDK (part of #186690) |
| ec37bf7 | Updated snapshot symbol assertions as a consequence of dart-lang/sdk@d36adba (cherry pick of #186812) |
| a46d1e8 | Same as above |
| 5a21bc3 | Fixes new `unawaited_return_in_try_block` analyzer errors caused by new Dart version (part of #186690) |
|03b6bf1 | Reverts unnecessary analysis errors I added earlier }

All of these fixes are included in this PR because their causes are a consequence of updating the Dart revision to the latest Dart beta (3.13.0-167.1.beta).

Part of #187787.

If this change needs to override an active code freeze, provide a comment explaining why. The code freeze workflow can be overridden by code reviewers. See pinned issues for any active code freezes with guidance.

**Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.
via-guy pushed a commit to via-guy/flutter that referenced this pull request Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD cp: beta cherry pick this pull request to beta release candidate branch engine flutter/engine related. See also e: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants