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

Issue 1583003: Change scm.GIT.CaptureStatus to ignore unmerged upstream changes. (Closed)

Created:
10 years, 8 months ago by Bernhard Bauer
Modified:
9 years, 7 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews
Visibility:
Public.

Description

Change scm.GIT.CaptureStatus to ignore unmerged upstream changes. With this CL, the patch that is sent to the code review site includes only changes from the nearest common ancestor of the local and the upstream branch. A diagram: /----B <- local branch ----A-----C <- upstream Previously, the diff would be simply between C and B, whereas now it's between A and C, even if you update C. This allows you to continously update origin/trunk without having to rebase/merge your local changes all the time, resulting in the need for less rebuilds. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=43113

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M scm.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Bernhard Bauer
10 years, 8 months ago (2010-03-30 14:56:46 UTC) #1
Evan Martin
What program is this for? Sorry, I have no context. :)
10 years, 8 months ago (2010-03-30 15:32:39 UTC) #2
Evan Martin
On 2010/03/30 15:32:39, Evan Martin wrote: > What program is this for? Sorry, I have ...
10 years, 8 months ago (2010-03-30 15:32:48 UTC) #3
Bernhard Bauer
On 2010/03/30 15:32:48, Evan Martin wrote: > On 2010/03/30 15:32:39, Evan Martin wrote: > > ...
10 years, 8 months ago (2010-03-30 16:49:03 UTC) #4
Evan Martin
On 2010/03/30 16:49:03, Bernhard Bauer wrote: > On 2010/03/30 15:32:48, Evan Martin wrote: > > ...
10 years, 8 months ago (2010-03-30 16:53:49 UTC) #5
Bernhard Bauer
10 years, 8 months ago (2010-04-09 10:20:29 UTC) #6
Heh, turns out it was *not* used in git-try. Another CL coming your way...

Powered by Google App Engine
This is Rietveld 408576698