Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(398)

Issue 1881223002: [Findit] Set culprit and infra failure explicitly. (Closed)

Created:
4 years, 8 months ago by stgao
Modified:
4 years, 8 months ago
Reviewers:
lijeffrey
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org, chanli, Sharu Jiang
Base URL:
https://chromium.googlesource.com/chromium/tools/build.git@master
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

[Findit] Set culprit and infra failure explicitly. If suspected revisions from heuristic analysis are passed over to the recipe, there might be more than one revision with compile failures. Thus set the culprit explicitly if found. BUG=chromium:596967 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299873

Patch Set 1 #

Messages

Total messages: 10 (6 generated)
stgao
4 years, 8 months ago (2016-04-12 21:43:10 UTC) #4
lijeffrey
lgtm
4 years, 8 months ago (2016-04-12 21:58:50 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1881223002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1881223002/1
4 years, 8 months ago (2016-04-12 22:01:18 UTC) #8
commit-bot: I haz the power
4 years, 8 months ago (2016-04-12 22:08:19 UTC) #10
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299873

Powered by Google App Engine
This is Rietveld 408576698