engine icon indicating copy to clipboard operation
engine copied to clipboard

Move `third_party/googletest` to `flutter/third_party`.

Open matanlurey opened this issue 1 year ago • 6 comments

As part of eliminating the Flutter buildroot (https://github.com/flutter/flutter/issues/67373), we are moving all third-party dependencies from //third_party to //flutter/third_party.

matanlurey avatar Dec 08 '23 19:12 matanlurey

Oof, this is failing on:

Using prebuilt Dart SDK binary. If you are editing Dart sources and wish to compile the Dart SDK, set `--no-prebuilt-dart-sdk`.
Generating GN files in: out/host_debug_unopt
ERROR at //third_party/abseil-cpp/absl.gni:109:7: Can't load input file.
      "//third_party/googletest:gmock",
      ^-------------------------------
Unable to load:
  /b/s/w/ir/cache/builder/src/third_party/googletest/BUILD.gn
I also checked in the secondary tree for:
  /b/s/w/ir/cache/builder/src/flutter/build/secondary/third_party/googletest/BUILD.gn

How do I edit https://flutter.googlesource.com/third_party/abseil-cpp/+/refs/heads/main/absl.gni?

matanlurey avatar Dec 08 '23 20:12 matanlurey

This pull request executed golden file tests, but it has not been updated in a while (20+ days). Test results from Gold expire after as many days, so this pull request will need to be updated with a fresh commit in order to get results from Gold.

flutter-dashboard[bot] avatar Jan 03 '24 19:01 flutter-dashboard[bot]

Converting to draft PR to keep it from coming up in triage.

cbracken avatar Jan 03 '24 19:01 cbracken

This pull request has been changed to a draft. The currently pending flutter-gold status will not be able to resolve until a new commit is pushed or the change is marked ready for review again.

flutter-dashboard[bot] avatar Jan 03 '24 19:01 flutter-dashboard[bot]

@matanlurey this is pretty old should this be closed or are you planning on working on it? (Desktop triage)

cbracken avatar Feb 15 '24 19:02 cbracken

I'll rebase this today. Thanks for the poke.

matanlurey avatar Feb 15 '24 19:02 matanlurey