commit | 0e6d0555d7ccc283df1443c9124177e4d44640a0 | [log] [tgz] |
---|---|---|
author | Sigurd Meldgaard <sigurdm@google.com> | Tue Dec 01 10:25:55 2020 +0000 |
committer | dart-luci-ci-builder <dart-luci-ci-builder@dart-ci.iam.gserviceaccount.com> | Tue Dec 01 02:27:35 2020 -0800 |
tree | 510648ceeee79a6ff507e5b2a93da978015a88f5 | |
parent | a693b10b667410971fa6ca12baac0b0939f72ad3 [diff] |
Improve handling of disable-dartdev-analytics This is second try of https://dart-review.googlesource.com/c/sdk/+/171284 that was reverted to to faulty logic in main_options. Some other refactorings are piggy-backed along. TestProject.runSync no longer takes a 'command' argument. It was anyway often not an argument. Also stop the messy handling of pub arguments. It is no longer needed. BUG: https://github.com/dart-lang/sdk/issues/44135 Change-Id: I49abf5810d9ea262409ba9d93f0471037cb8a753 TEST=The VM change is tested via all the pkg/dartdev/test/command/* tests that invoke dart with the --no-analytics flag. TEST=Furthermore manual test that the --no-analytics flag is passed to dartdev. Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/174261 Reviewed-by: Jonas Jensen <jonasfj@google.com> Commit-Queue: Sigurd Meldgaard <sigurdm@google.com> https://dart.googlesource.com/sdk/+/58860f4814fb8bd36d0ebd580425a773ea7ede8c
diff --git a/commits.json b/commits.json index c4c2144..5e2327a 100644 --- a/commits.json +++ b/commits.json
@@ -1,5 +1,5 @@ { "external/github.com/flutter/engine":"a4ce848dfb6fb002544494e762f53a826830cc1d", "external/github.com/flutter/flutter":"a7f5fd5360007518644597c60a1f2169eacccc2b", - "sdk":"deca91c86d9f2a70c7d6fc6d9f594715d361a110" + "sdk":"58860f4814fb8bd36d0ebd580425a773ea7ede8c" } \ No newline at end of file