Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Infer --rbe based on the existence of //flutter/build/rbe#52700

Merged
matanlurey merged 3 commits into
flutter-team-archive:mainfrom
matanlurey:et-rbe
May 10, 2024
Merged

Infer --rbe based on the existence of //flutter/build/rbe#52700
matanlurey merged 3 commits into
flutter-team-archive:mainfrom
matanlurey:et-rbe

Conversation

@matanlurey

Copy link
Copy Markdown
Contributor

@cbracken cbracken 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.

lgtm but sad lints

Animated GIF of a delivery truck approaching a highway overpass, then failing to meet the height clearance requirement and crashing into it

@zanderso zanderso 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.

This lgtm, but could you confirm whether there's a test that will cover the new behavior of failing when --rbe is supplied but the directory doesn't exist?

@matanlurey matanlurey merged commit 254927e into flutter-team-archive:main May 10, 2024
@matanlurey matanlurey deleted the et-rbe branch May 10, 2024 18:49
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request May 10, 2024
flutter-team-archive/engine@d4f705c...ba8e0d3

2024-05-10 1961493+harryterkelsen@users.noreply.github.com [canvaskit] Clip before applying ColorFilter so it doesn't filter beyond child bounds (flutter-team-archive/engine#52704)
2024-05-10 magder@google.com Migrate FlutterView, FlutterPlatformViews, FlutterOverlayView to ARC (flutter-team-archive/engine#52535)
2024-05-10 matanlurey@users.noreply.github.com Infer `--rbe` based on the existence of `//flutter/build/rbe` (flutter-team-archive/engine#52700)
2024-05-10 jonahwilliams@google.com [Impeller] Disable AHB swapchain. (flutter-team-archive/engine#52713)
2024-05-10 skia-flutter-autoroll@skia.org Roll Skia from c7cd1e9690d1 to 11d892ce49b6 (25 revisions) (flutter-team-archive/engine#52712)
2024-05-10 chinmaygarde@google.com [Impeller] Document how to do basic rendering in Impeller. (flutter-team-archive/engine#52703)
2024-05-10 30870216+gaaclarke@users.noreply.github.com [impeller] adds experimental canvas docstring (flutter-team-archive/engine#52710)
2024-05-10 daniel.l@hpcnt.com Roll third_party/freetype2 from 3bea27612 to af4c2d86d (2 revisions) (flutter-team-archive/engine#52689)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: 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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

[et] --rbe should always succeed, or fail if it is not available

3 participants