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

Issue 661413005: Make 'git cl patch' work when not on a named branch. (Closed)

Created:
6 years, 2 months ago by szager1
Modified:
6 years, 2 months ago
Reviewers:
M-A Ruel, pgervais
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, iannucci+depot_tools_chromium.org
Project:
tools
Visibility:
Public.

Description

Make 'git cl patch' work when not on a named branch. R=maruel@chromium.org,pgervais@chromium.org BUG= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=292537

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -4 lines) Patch
M git_cl.py View 3 chunks +13 lines, -4 lines 2 comments Download

Messages

Total messages: 9 (1 generated)
szager1
6 years, 2 months ago (2014-10-20 20:38:42 UTC) #1
M-A Ruel
That's not a use case I had expected but 'ok'. lgtm
6 years, 2 months ago (2014-10-20 20:43:34 UTC) #2
pgervais
I have one nit. https://codereview.chromium.org/661413005/diff/1/git_cl.py File git_cl.py (right): https://codereview.chromium.org/661413005/diff/1/git_cl.py#newcode532 git_cl.py:532: self.branchref = branchref Please define ...
6 years, 2 months ago (2014-10-20 20:45:57 UTC) #3
pgervais
I'm also wondering whether this could have side-effects on bots.
6 years, 2 months ago (2014-10-20 20:46:51 UTC) #4
szager1
On 2014/10/20 20:43:34, M-A Ruel wrote: > That's not a use case I had expected ...
6 years, 2 months ago (2014-10-20 20:55:58 UTC) #5
szager1
https://codereview.chromium.org/661413005/diff/1/git_cl.py File git_cl.py (right): https://codereview.chromium.org/661413005/diff/1/git_cl.py#newcode532 git_cl.py:532: self.branchref = branchref On 2014/10/20 20:45:57, pgervais wrote: > ...
6 years, 2 months ago (2014-10-20 22:29:52 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/661413005/1
6 years, 2 months ago (2014-10-20 22:31:17 UTC) #8
commit-bot: I haz the power
6 years, 2 months ago (2014-10-20 22:33:23 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 292537

Powered by Google App Engine
This is Rietveld 408576698