Skip to content

Use release-*.version to simplify last_engine_commit.sh (no branch operations) #172236

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
merged 1 commit into from
Jul 16, 2025

Conversation

matanlurey
Copy link
Contributor

This fixes #172184 by considering the last commit to release-candidate-branch.version as the significant commit.

Otherwise, operationally, it works the same. I tested this on 3.35 and it works as expected (where the current does not).

@matanlurey matanlurey requested a review from jtmcdole July 16, 2025 17:01
@matanlurey matanlurey added the emergency Jump the queue; land PR in front of all others; only use for emergencies label Jul 16, 2025
@flutter-dashboard
Copy link

Detected the emergency label.

If you add the autosubmit label, the bot will wait until all presubmits pass but ignore the tree status, allowing fixes for tree breakages while still validating that they don't break any existing presubmits.

The "Merge" button is also unlocked. To bypass presubmits as well as the tree status, press the GitHub "Add to Merge Queue".

@matanlurey
Copy link
Contributor Author

Landing as emergency to reduce the time to cherry-pick this into the 3.35 branch.

@matanlurey matanlurey added this pull request to the merge queue Jul 16, 2025
Merged via the queue into flutter:master with commit 0669195 Jul 16, 2025
147 of 150 checks passed
@matanlurey matanlurey deleted the simplify-last_engine_commit branch July 16, 2025 17:42
auto-submit bot pushed a commit that referenced this pull request Jul 16, 2025
Cherrypick's #172236 into the 3.35 release branch _and_ tests it live by setting `engine.version` in this PR.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 17, 2025
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jul 17, 2025
flutter/flutter@c2739f0...9c626d9

2025-07-17 [email protected] Roll Skia from 5b4afa58289a to 3673a1f26a63 (1 revision) (flutter/flutter#172274)
2025-07-17 [email protected] Roll Fuchsia Linux SDK from JRFUXSNXExcfjVYvA... to HXdm7P0a4ZJVm_TE-... (flutter/flutter#172268)
2025-07-17 [email protected] Roll Dart SDK from d71df90177e4 to 486f9c0663bc (1 revision) (flutter/flutter#172269)
2025-07-17 [email protected] Roll Skia from 3ad1aace3e02 to 5b4afa58289a (3 revisions) (flutter/flutter#172264)
2025-07-17 [email protected] Perform OpenGL compositing in the Flutter thread and write to a framebuffer. (flutter/flutter#172090)
2025-07-17 [email protected] Licenses cpp 716 (flutter/flutter#172261)
2025-07-16 [email protected] Remove emoji from ci.yaml, because we still live with CP1252 for some silly reason (flutter/flutter#172256)
2025-07-16 [email protected] Roll Skia from bf3f9b77d3a8 to 3ad1aace3e02 (11 revisions) (flutter/flutter#172255)
2025-07-16 [email protected] Sync `CHANGELOG.md` (3.32.7) to `master` branch (flutter/flutter#172253)
2025-07-16 [email protected] Properly lay out and position RenderWebImage (flutter/flutter#171916)
2025-07-16 [email protected] Remove dead link in the doc index. (flutter/flutter#172240)
2025-07-16 [email protected] Roll Dart SDK from 766ee8029b11 to d71df90177e4 (1 revision) (flutter/flutter#172243)
2025-07-16 [email protected] Fix use of `Join-Path` in `last_engine_commit.ps1`(poweshell) (flutter/flutter#172242)
2025-07-16 [email protected] Marks Linux_pixel_7pro integration_ui_driver to be unflaky (flutter/flutter#172213)
2025-07-16 [email protected] [skwasm] Decrease reliance on finalizers/GC (flutter/flutter#172187)
2025-07-16 [email protected] Use `release-*.version` to simplify `last_engine_commit.sh` (no branch operations) (flutter/flutter#172236)
2025-07-16 [email protected] Roll Skia from 59be8479c637 to bf3f9b77d3a8 (7 revisions) (flutter/flutter#172233)
2025-07-16 [email protected] [web] Remove all usages of js_util. (flutter/flutter#171871)

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
emergency Jump the queue; land PR in front of all others; only use for emergencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants