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

Issue 930693002: gclient: Add '--auto_rebase' sync option. (Closed)

Created:
5 years, 10 months ago by dnj
Modified:
5 years, 10 months ago
Reviewers:
iannucci, hinoka
CC:
chromium-reviews, cmp-cc_chromium.org, dpranke+depot_tools_chromium.org, iannucci+depot_tools_chromium.org
Target Ref:
refs/heads/master
Project:
tools
Visibility:
Public.

Description

gclient: Add '--auto_rebase' sync option. This adds the '--auto_rebase' sync option, enabling parallel updates to automatically rebase local Git branches during sync. BUG=None TEST=local Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=294151

Patch Set 1 #

Total comments: 2

Patch Set 2 : Updated argument to have an underscore. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -9 lines) Patch
M gclient.py View 1 1 chunk +3 lines, -0 lines 0 comments Download
M gclient_scm.py View 1 chunk +10 lines, -9 lines 0 comments Download
M tests/gclient_scm_test.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 10 (2 generated)
dnj
Got sick of "-j1" on mostly-ToT gclient repository set that needed straightforward rebasing. Now I ...
5 years, 10 months ago (2015-02-15 17:52:55 UTC) #2
hinoka
Seems useful to me too! Maybe even make this default behavior and only give up ...
5 years, 10 months ago (2015-02-17 18:43:56 UTC) #3
dnj
I'm reluctant to make it the default behavior, as it can potentially negatively impact the ...
5 years, 10 months ago (2015-02-17 19:57:59 UTC) #4
dnj
Ping! I've used this a few times and found it useful!
5 years, 10 months ago (2015-02-20 21:19:11 UTC) #5
iannucci
oh! for some reason I thought this already had an lgtm.
5 years, 10 months ago (2015-02-20 21:20:20 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/930693002/20001
5 years, 10 months ago (2015-02-20 21:22:53 UTC) #8
dnj
Thanks :)
5 years, 10 months ago (2015-02-20 21:22:54 UTC) #9
commit-bot: I haz the power
5 years, 10 months ago (2015-02-20 21:25:59 UTC) #10
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=294151

Powered by Google App Engine
This is Rietveld 408576698