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

Issue 10825409: me2me_test_dependencies (Closed)

Created:
8 years, 4 months ago by yihongg1
Modified:
8 years, 4 months ago
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

- Modified FILES.cfg to copy remoting host to archive. - Added remoting host dependencies in chromium_builder_qa so that chromium bot will have remoting host to support me2me test cases. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=152222

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -2 lines) Patch
M build/all.gyp View 1 1 chunk +10 lines, -0 lines 0 comments Download
M chrome/tools/build/mac/FILES.cfg View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/tools/build/win/FILES.cfg View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 11 (0 generated)
yihongg
This for enabling me2me pyauto automation.
8 years, 4 months ago (2012-08-17 00:21:09 UTC) #1
Nirnimesh
LGTM
8 years, 4 months ago (2012-08-17 00:24:38 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yihongg@chromium.org/10825409/1
8 years, 4 months ago (2012-08-17 17:40:50 UTC) #3
commit-bot: I haz the power
Presubmit check for 10825409-1 failed and returned exit status 1. Running presubmit commit checks ...
8 years, 4 months ago (2012-08-17 17:40:58 UTC) #4
yihongg
Kerz, could you review and LGTM this?
8 years, 4 months ago (2012-08-17 17:44:00 UTC) #5
kerz_chromium
lgtm
8 years, 4 months ago (2012-08-17 17:45:46 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yihongg@chromium.org/10825409/1
8 years, 4 months ago (2012-08-17 17:48:49 UTC) #7
commit-bot: I haz the power
Try job failure for 10825409-1 (retry) on win for step "runhooks". It's a second try, ...
8 years, 4 months ago (2012-08-17 18:22:36 UTC) #8
yihongg
8 years, 4 months ago (2012-08-17 22:26:17 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yihongg@chromium.org/10825409/1004
8 years, 4 months ago (2012-08-17 22:26:53 UTC) #10
commit-bot: I haz the power
8 years, 4 months ago (2012-08-18 02:26:16 UTC) #11
Change committed as 152222

Powered by Google App Engine
This is Rietveld 408576698