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

Correct the Fuchsia licenses golden file #16215

Merged
merged 1 commit into from
Jan 30, 2020

Conversation

cbracken
Copy link
Member

@cbracken cbracken commented Jan 30, 2020

Looks like this was triggered by the recent tonic deletion.

Related commits:

This is preventing the Dart roller from rolling.

This is preventing the Dart roller from rolling.
@cbracken
Copy link
Member Author

cbracken commented Jan 30, 2020

FWIW this looks like a bug in the signature-based skipping we do in the license tool. Looking at the output of the Cirrus run of the format_and_dart_test on #15895, I see:

Collecting licenses for fuchsia
    Skipping this component - no change in signature
Collecting licenses for third_party
    Skipping this component - no change in signature
Collecting licenses for skia
    Skipping this component - no change in signature
Verifying license tool signature...
Checking license count in licenses_flutter...
Licenses are as expected.

Will file a bug and take a quick look this week.

@cbracken cbracken merged commit b98eadc into flutter:master Jan 30, 2020
@cbracken cbracken deleted the fix-licenses branch January 30, 2020 01:59
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 30, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 30, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 30, 2020
flutter/engine@6061b80...f6526c0

git log 6061b80..f6526c0 --first-parent --oneline
2020-01-30 [email protected] Roll src/third_party/skia 25bcd6a6215e..8f44c3504ca6 (2 commits) (flutter/engine#16243)
2020-01-30 [email protected] Roll src/third_party/skia 24ed75db12ba..25bcd6a6215e (5 commits) (flutter/engine#16234)
2020-01-30 [email protected] Make output of run_tests.py easier to understand. (flutter/engine#16229)
2020-01-30 [email protected] Roll fuchsia/sdk/core/mac-amd64 from c3sk5... to MK0YU... (flutter/engine#16221)
2020-01-30 [email protected] Remove buggy test to check if tasks can be posted on all native threads recursively from the emebdder. (flutter/engine#16214)
2020-01-30 [email protected] Correct the Fuchsia licenses golden file (flutter/engine#16215)
2020-01-30 [email protected] Roll src/third_party/skia a2a0c8a2c475..24ed75db12ba (12 commits) (flutter/engine#16216)


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 [email protected] on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
stuartmorgan-g pushed a commit to stuartmorgan-g/engine that referenced this pull request Jan 30, 2020
Looks like this was triggered by the recent tonic deletion.

Related commits:
* Move tonic into //flutter/third_party. flutter#15895
* Remove tonic/platform. flutter#16062

This is preventing the Dart roller from rolling.
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
Looks like this was triggered by the recent tonic deletion.

Related commits:
* Move tonic into //flutter/third_party. flutter#15895
* Remove tonic/platform. flutter#16062

This is preventing the Dart roller from rolling.
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants