# [PATCH] doc/rebase: extend examples to show continuing branches

3 messages from 2019-03-05 to 2019-03-05. Participants: Robert P. J. Day, Junio C Hamano.
Thread: https://gitlist.dev/t/50656

## Robert P. J. Day, 2019-03-05 10:48

Subject: [PATCH] doc/rebase: extend examples to show continuing branches
Message-ID: <alpine.LFD.2.21.1903050545460.24324@localhost.localdomain>
URL: https://gitlist.dev/e/alpine.LFD.2.21.1903050545460.24324%40localhost.localdomain

```

Currently, all of the examples for "man git-rebase" show rebasing from
a branch that has had no further development, which might mislead
readers into thinking that that is a necessary condition for rebasing,
so tweak the examples to show further development on such a branch to
clarify that.

Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---

diff --git a/Documentation/git-rebase.txt b/Documentation/git-rebase.txt
index 5629ba4c5d..ba3c44fdaf 100644
--- a/Documentation/git-rebase.txt
+++ b/Documentation/git-rebase.txt
@@ -109,8 +109,8 @@ functionality which is found in 'next'.
     o---o---o---o---o  master
          \
           o---o---o---o---o  next
-                           \
-                            o---o---o  topic
+                   \
+                    o---o---o  topic
 ------------

 We want to make 'topic' forked from branch 'master'; for example,
@@ -134,8 +134,8 @@ Another example of --onto option is to rebase part of a
 branch.  If we have the following situation:

 ------------
-                            H---I---J topicB
-                           /
+                        H---I---J topicB
+                       /
                   E---F---G  topicA
                  /
     A---B---C---D  master

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                  http://crashcourse.ca/dokuwiki

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================

```

## Junio C Hamano, 2019-03-05 14:03

Subject: Re: [PATCH] doc/rebase: extend examples to show continuing branches
Message-ID: <xmqqr2blto3k.fsf@gitster-ct.c.googlers.com>
URL: https://gitlist.dev/e/xmqqr2blto3k.fsf%40gitster-ct.c.googlers.com
In-Reply-To: <alpine.LFD.2.21.1903050545460.24324@localhost.localdomain>

```
"Robert P. J. Day" <rpjday@crashcourse.ca> writes:

> Currently, all of the examples for "man git-rebase" show rebasing from
> a branch that has had no further development, which might mislead
> readers into thinking that that is a necessary condition for rebasing,
> so tweak the examples to show further development on such a branch to
> clarify that.

We state the status-quo in present tense to start problem
description, so "Currently" is a noise word you can and should omit.

As I already said, at least one example that rebases a branch that
was forked from the midpoint of another branch, so the problem
description is already false.  If we apply this patch, do we lose
all examples that rebase a branch that purely builds on top of the
tip of another branch?  That would also mislead readers into
thinking that you need to advance the base branch before you can
rebase the forked branch ;-).


```

## Robert P. J. Day, 2019-03-05 14:04

Subject: Re: [PATCH] doc/rebase: extend examples to show continuing branches
Message-ID: <alpine.LFD.2.21.1903050904350.30374@localhost.localdomain>
URL: https://gitlist.dev/e/alpine.LFD.2.21.1903050904350.30374%40localhost.localdomain
In-Reply-To: <xmqqr2blto3k.fsf@gitster-ct.c.googlers.com>

```
On Tue, 5 Mar 2019, Junio C Hamano wrote:

> "Robert P. J. Day" <rpjday@crashcourse.ca> writes:
>
> > Currently, all of the examples for "man git-rebase" show rebasing from
> > a branch that has had no further development, which might mislead
> > readers into thinking that that is a necessary condition for rebasing,
> > so tweak the examples to show further development on such a branch to
> > clarify that.
>
> We state the status-quo in present tense to start problem
> description, so "Currently" is a noise word you can and should omit.
>
> As I already said, at least one example that rebases a branch that
> was forked from the midpoint of another branch, so the problem
> description is already false.  If we apply this patch, do we lose
> all examples that rebase a branch that purely builds on top of the
> tip of another branch?  That would also mislead readers into
> thinking that you need to advance the base branch before you can
> rebase the forked branch ;-).

  i stand corrected. carry on. :-)

rday

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                  http://crashcourse.ca/dokuwiki

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================

```
