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

re-land "update the repo references for package:file (#55906)"#55935

Merged
devoncarew merged 4 commits into
mainfrom
revert-55916-revert_066a3c71187bb3b6ef694decb9c32287bcd06e45
Oct 21, 2024
Merged

re-land "update the repo references for package:file (#55906)"#55935
devoncarew merged 4 commits into
mainfrom
revert-55916-revert_066a3c71187bb3b6ef694decb9c32287bcd06e45

Conversation

@devoncarew

Copy link
Copy Markdown
Contributor

This is a re-land of the reverted PR, #55916 (it depended on a dart roll that was reverted, and has subsequently been re-landed).

From the original PR:

The SoT repository for package:file has changed from google/file.dart to dart-lang/tools; this PR updates the DEPS file and the hard-coded paths in the two pubspec.yaml files (see dart-lang/tools#674).

@github-actions github-actions Bot added the platform-web Code specifically for the web engine label Oct 17, 2024
@devoncarew
devoncarew requested a review from zanderso October 18, 2024 00:05

@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 should wait until the internal roll succeeds since we may need to roll back again.

@devoncarew

Copy link
Copy Markdown
Contributor Author

This should wait until the internal roll succeeds since we may need to roll back again.

Gotcha - I'll hold off on landing this for a bit.

@devoncarew
devoncarew requested a review from zanderso October 18, 2024 14:54

@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 is good to go

@devoncarew
devoncarew merged commit de569ba into main Oct 21, 2024
@devoncarew
devoncarew deleted the revert-55916-revert_066a3c71187bb3b6ef694decb9c32287bcd06e45 branch October 21, 2024 17:39
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Oct 21, 2024
flutter-team-archive/engine@315540e...de569ba

2024-10-21 devoncarew@google.com re-land "update the repo references for package:file (#55906)" (flutter-team-archive/engine#55935)

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 jonahwilliams@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
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
…flutter-team-archive/engine#55935)

This is a re-land of the reverted PR, flutter-team-archive/engine#55916 (it depended
on a dart roll that was reverted, and has subsequently been re-landed).

From the original PR:

> The SoT repository for `package:file` has changed from
google/file.dart to dart-lang/tools; this PR updates the DEPS file and
the hard-coded paths in the two pubspec.yaml files (see
dart-lang/tools#674).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

platform-web Code specifically for the web engine

Development

Successfully merging this pull request may close these issues.

2 participants