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

Issue 920493002: Run gn_unittests on gn_upload step. (Closed)

Created:
5 years, 10 months ago by tfarina
Modified:
5 years, 10 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews, kjellander-cc_chromium.org, cmp-cc_chromium.org, stip+watch_chromium.org, brettw, luqui, M-A Ruel, scottmg
Target Ref:
refs/heads/master
Project:
tools
Visibility:
Public.

Description

Run gn_unittests on gn_upload step. After the fix in https://chromium.googlesource.com/chromium/src.git/+/7f1ee75851d2a0f19ce8fe993cd7e9e1ccfebf4e we should be able to run gn_unittests from anywhere, not just from the root of the checkout. BUG=443813 R=dpranke@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=294040

Patch Set 1 #

Patch Set 2 : ./scripts/slave/unittests/recipe_simulation_test.py train #

Messages

Total messages: 10 (4 generated)
tfarina
5 years, 10 months ago (2015-02-11 13:07:10 UTC) #1
Dirk Pranke
lgtm
5 years, 10 months ago (2015-02-11 17:22:24 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/920493002/1
5 years, 10 months ago (2015-02-12 12:43:50 UTC) #4
commit-bot: I haz the power
Presubmit check for 920493002-1 failed and returned exit status 1. Running presubmit commit checks ...
5 years, 10 months ago (2015-02-12 12:46:41 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/920493002/20001
5 years, 10 months ago (2015-02-12 13:02:52 UTC) #9
commit-bot: I haz the power
5 years, 10 months ago (2015-02-12 13:05:46 UTC) #10
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=294040

Powered by Google App Engine
This is Rietveld 408576698