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

Issue 113553003: Roll swarm_client @ 9fc05789e3. (Closed)

Created:
7 years ago by M-A Ruel
Modified:
7 years ago
Reviewers:
Vadim Sh.
CC:
chromium-reviews, ganetsky1
Visibility:
Public.

Description

Roll swarm_client @ 9fc05789e3. Includes revert of r240243. $ git log aab3a62beb..9fc05789e3 --date=short --format="%ad %ae %s" | sed 's/@chromium\.org//' 2013-12-11 ganetsky Create a .gitignore file and add *.pyc to it 2013-12-11 maruel Stop signalling swarming error on Windows file locking. 2013-12-11 maruel Ensure variables are decoded to unicode. 2013-12-06 maruel Rename --command-variable to --extra-variables. 2013-12-06 maruel Add real variable support and split --variable in 3 different flags. 2013-12-05 vadimsh Add 'offset' parameter to StorageApi.fetch(...) method. 2013-12-05 maruel Change log message to be clearer. 2013-12-05 vadimsh Support custom HTTP headers in net.url_open. 2013-12-03 vadimsh Remove urllib2 support. 2013-12-03 vadimsh Wait for Storage's internal threads to finish before exiting. 2013-12-03 vadimsh Fix swarming_smoke_test.py and isolateserver_test.py. R=vadimsh@chromium.org NOTRY=true BUG= Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=240460

Patch Set 1 #

Patch Set 2 : Now include revert of r240243 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -7 lines) Patch
M DEPS View 1 1 chunk +1 line, -1 line 0 comments Download
M build/android/pylib/gtest/setup.py View 1 1 chunk +2 lines, -2 lines 0 comments Download
M build/isolate.gypi View 1 4 chunks +8 lines, -4 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
M-A Ruel
7 years ago (2013-12-12 02:19:54 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/113553003/1
7 years ago (2013-12-12 02:22:36 UTC) #2
Vadim Sh.
lgtm
7 years ago (2013-12-12 02:27:01 UTC) #3
Vadim Sh.
On 2013/12/12 02:27:01, Vadim Sh. wrote: > lgtm I unchecked CQ box, previous roll was ...
7 years ago (2013-12-12 03:14:14 UTC) #4
M-A Ruel
On 2013/12/12 03:14:14, Vadim Sh. wrote: > On 2013/12/12 02:27:01, Vadim Sh. wrote: > > ...
7 years ago (2013-12-12 15:46:34 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/113553003/20001
7 years ago (2013-12-12 16:25:26 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/113553003/110001
7 years ago (2013-12-12 19:15:10 UTC) #7
M-A Ruel
7 years ago (2013-12-12 22:59:39 UTC) #8
Message was sent while issue was closed.
Committed patchset #2 manually as r240460 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698