Chromium Code Reviews
DescriptionReland of Add a library for running external commands, providing timeouts and test injection. (patchset #1 id:1 of https://codereview.chromium.org/1295043002/ )
Reason for revert:
Fixed bug in original code - see patchset #2.
Original issue's description:
> Revert of Add a library for running external commands, providing timeouts and test injection. (patchset #2 id:40001 of https://codereview.chromium.org/1290913004/ )
>
> Reason for revert:
> CT Chromium Perf tasks are failing with:
>
> E0815 11:07:22.649344 13091 exec.go:193] Command exited with invalid argument: python /b/skia-repo/go/src/go.skia.org/infra/ct/py/sync_skia_in_chrome.py --destination=/b/storage/chromium_builds/linux_base --fetch_target=chromium --chrome_revision=6ae54795f6d077e31f4fb42540ead46f4141f2bb --skia_revision=fb28cd2b13d76c324570341e1155aaadc1b532b6
>
> Original issue's description:
> > Add a library for running external commands, providing timeouts and test injection.
> >
> > BUG=skia:4106
> >
> > Committed: https://skia.googlesource.com/buildbot/+/b65fde6749a3e060938cd280c7e281e29516edc1
>
> TBR=jcgregorio@google.com,benjaminwagner@google.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:4106
>
> Committed: https://skia.googlesource.com/buildbot/+/ac6c1292955c888d35f6c01cf97d3b6ff4029424
BUG=skia:4106
Committed: https://skia.googlesource.com/buildbot/+/a12e89205d4145b4da14b0a7d5b95b15b029ada9
Patch Set 1 #Patch Set 2 : Fix bug with nil io.Writer. #
Messages
Total messages: 9 (2 generated)
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||