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

Issue 9233057: Update upload.py @ fae51921ad9d (Closed)

Created:
8 years, 10 months ago by M-A Ruel
Modified:
8 years, 10 months ago
Reviewers:
cmp
CC:
chromium-reviews, Dirk Pranke
Visibility:
Public.

Description

Update upload.py @ fae51921ad9d This version of upload.py is slightly less annoying, it won't cancel out if no patchset title is provided and disable colors all the time. R=cmp@chromium.org BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=121826

Patch Set 1 #

Patch Set 2 : gcl is now a happier place #

Total comments: 1

Patch Set 3 : . #

Patch Set 4 : fae51921ad9d is now the chill revision #

Patch Set 5 : Rebase against HEAD #

Unified diffs Side-by-side diffs Delta from patch set Stats (+122 lines, -134 lines) Patch
M gcl.py View 1 5 chunks +8 lines, -27 lines 0 comments Download
M git_cl.py View 1 2 3 4 4 chunks +25 lines, -34 lines 0 comments Download
M tests/gcl_unittest.py View 1 2 6 chunks +7 lines, -8 lines 0 comments Download
M tests/git_cl_test.py View 1 2 3 4 1 chunk +1 line, -3 lines 0 comments Download
M third_party/upload.py View 1 2 3 19 chunks +81 lines, -62 lines 0 comments Download

Messages

Total messages: 25 (0 generated)
M-A Ruel
See the diff between patchsets.
8 years, 10 months ago (2012-01-31 16:33:52 UTC) #1
cmp
lgtm assuming no change in user experience Let me know when you land this.
8 years, 10 months ago (2012-01-31 21:48:04 UTC) #2
cmp
https://chromiumcodereview.appspot.com/9233057/diff/2001/tests/gcl_unittest.py File tests/gcl_unittest.py (right): https://chromiumcodereview.appspot.com/9233057/diff/2001/tests/gcl_unittest.py#newcode316 tests/gcl_unittest.py:316: '--issue=1', '--title= '], nit: remove one of the spaces ...
8 years, 10 months ago (2012-01-31 21:48:24 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/12001
8 years, 10 months ago (2012-02-01 16:04:14 UTC) #4
commit-bot: I haz the power
Presubmit check for 9233057-12001 failed and returned exit status 1. warning: code.google.com certificate with fingerprint ...
8 years, 10 months ago (2012-02-01 16:10:00 UTC) #5
M-A Ruel
On 2012/02/01 16:10:00, I haz the power (commit-bot) wrote: > Presubmit check for 9233057-12001 failed ...
8 years, 10 months ago (2012-02-01 16:26:30 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/9233057/12001
8 years, 10 months ago (2012-02-01 16:26:45 UTC) #7
commit-bot: I haz the power
Presubmit check for 9233057-12001 failed and returned exit status 1. warning: code.google.com certificate with fingerprint ...
8 years, 10 months ago (2012-02-01 16:32:29 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/12001
8 years, 10 months ago (2012-02-01 18:11:34 UTC) #9
commit-bot: I haz the power
Presubmit check for 9233057-12001 failed and returned exit status 1. warning: code.google.com certificate with fingerprint ...
8 years, 10 months ago (2012-02-01 18:17:16 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/16001
8 years, 10 months ago (2012-02-13 21:06:37 UTC) #11
commit-bot: I haz the power
Can't apply patch for file depot_tools/third_party/upload.py. While running patch -p1 --forward --force; patching file depot_tools/third_party/upload.py ...
8 years, 10 months ago (2012-02-13 21:06:39 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/21001
8 years, 10 months ago (2012-02-13 23:36:29 UTC) #13
commit-bot: I haz the power
Can't apply patch for file depot_tools/third_party/upload.py. While running patch -p1 --forward --force; patching file depot_tools/third_party/upload.py ...
8 years, 10 months ago (2012-02-13 23:36:39 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/21001
8 years, 10 months ago (2012-02-14 00:35:27 UTC) #15
commit-bot: I haz the power
Can't apply patch for file depot_tools/third_party/upload.py. While running patch -p1 --forward --force; patching file depot_tools/third_party/upload.py ...
8 years, 10 months ago (2012-02-14 00:35:30 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/21001
8 years, 10 months ago (2012-02-14 00:36:25 UTC) #17
commit-bot: I haz the power
Commit queue had an internal error. Something went really wrong, probably a crash, a hickup ...
8 years, 10 months ago (2012-02-14 00:36:33 UTC) #18
M-A Ruel
On 2012/02/14 00:36:33, I haz the power (commit-bot) wrote: > Commit queue had an internal ...
8 years, 10 months ago (2012-02-14 00:38:30 UTC) #19
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/21001
8 years, 10 months ago (2012-02-14 00:38:42 UTC) #20
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/21001
8 years, 10 months ago (2012-02-14 00:41:45 UTC) #21
commit-bot: I haz the power
Can't apply patch for file depot_tools/third_party/upload.py. While running patch -p1 --forward --force; patching file depot_tools/third_party/upload.py ...
8 years, 10 months ago (2012-02-14 00:41:49 UTC) #22
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9233057/21001
8 years, 10 months ago (2012-02-14 02:27:51 UTC) #23
commit-bot: I haz the power
Change committed as 121826
8 years, 10 months ago (2012-02-14 02:34:17 UTC) #24
M-A Ruel
8 years, 10 months ago (2012-02-14 02:35:41 UTC) #25
FTR, it needed https://chromiumcodereview.appspot.com/9387024/ to fix the
unicode upgrade issue.

Powered by Google App Engine
This is Rietveld 408576698