{"thread":{"id":"50656","subject":"[PATCH] doc/rebase: extend examples to show continuing branches","startedAt":"2019-03-05T10:48:34Z","lastAt":"2019-03-05T14:05:06Z","messageCount":3,"participants":["Robert P. J. Day","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"370676","messageId":"alpine.LFD.2.21.1903050545460.24324@localhost.localdomain","threadId":"50656","inReplyTo":null,"subject":"[PATCH] doc/rebase: extend examples to show continuing branches","fromName":"Robert P. J. Day","fromEmail":"rpjday@crashcourse.ca","sentAt":"2019-03-05T10:48:28Z","receivedAt":"2019-03-05T10:48:34Z","isPatch":true,"sender":{"key":"rpjday@crashcourse.ca","avatar":"https://avatars.githubusercontent.com/u/226084077?v=4"},"body":"\nCurrently, all of the examples for \"man git-rebase\" show rebasing from\na branch that has had no further development, which might mislead\nreaders into thinking that that is a necessary condition for rebasing,\nso tweak the examples to show further development on such a branch to\nclarify that.\n\nSigned-off-by: Robert P. J. Day \n\n---\n\ndiff --git a/Documentation/git-rebase.txt b/Documentation/git-rebase.txt\nindex 5629ba4c5d..ba3c44fdaf 100644\n--- a/Documentation/git-rebase.txt\n+++ b/Documentation/git-rebase.txt\n@@ -109,8 +109,8 @@ functionality which is found in 'next'.\n o---o---o---o---o master\n \\\n o---o---o---o---o next\n- \\\n- o---o---o topic\n+ \\\n+ o---o---o topic\n ------------\n\n We want to make 'topic' forked from branch 'master'; for example,\n@@ -134,8 +134,8 @@ Another example of --onto option is to rebase part of a\n branch. If we have the following situation:\n\n ------------\n- H---I---J topicB\n- /\n+ H---I---J topicB\n+ /\n E---F---G topicA\n /\n A---B---C---D master\n\n-- \n\n========================================================================\nRobert P. J. Day Ottawa, Ontario, CANADA\n http://crashcourse.ca/dokuwiki\n\nTwitter: http://twitter.com/rpjday\nLinkedIn: http://ca.linkedin.com/in/rpjday\n========================================================================\n"},{"id":"370716","messageId":"xmqqr2blto3k.fsf@gitster-ct.c.googlers.com","threadId":"50656","inReplyTo":"alpine.LFD.2.21.1903050545460.24324@localhost.localdomain","subject":"Re: [PATCH] doc/rebase: extend examples to show continuing branches","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-03-05T14:03:43Z","receivedAt":"2019-03-05T14:03:48Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Robert P. J. Day\" writes:\n\n> Currently, all of the examples for \"man git-rebase\" show rebasing from\n> a branch that has had no further development, which might mislead\n> readers into thinking that that is a necessary condition for rebasing,\n> so tweak the examples to show further development on such a branch to\n> clarify that.\n\nWe state the status-quo in present tense to start problem\ndescription, so \"Currently\" is a noise word you can and should omit.\n\nAs I already said, at least one example that rebases a branch that\nwas forked from the midpoint of another branch, so the problem\ndescription is already false. If we apply this patch, do we lose\nall examples that rebase a branch that purely builds on top of the\ntip of another branch? That would also mislead readers into\nthinking that you need to advance the base branch before you can\nrebase the forked branch ;-).\n\n"},{"id":"370718","messageId":"alpine.LFD.2.21.1903050904350.30374@localhost.localdomain","threadId":"50656","inReplyTo":"xmqqr2blto3k.fsf@gitster-ct.c.googlers.com","subject":"Re: [PATCH] doc/rebase: extend examples to show continuing branches","fromName":"Robert P. J. Day","fromEmail":"rpjday@crashcourse.ca","sentAt":"2019-03-05T14:04:56Z","receivedAt":"2019-03-05T14:05:06Z","isPatch":true,"sender":{"key":"rpjday@crashcourse.ca","avatar":"https://avatars.githubusercontent.com/u/226084077?v=4"},"body":"On Tue, 5 Mar 2019, Junio C Hamano wrote:\n\n> \"Robert P. J. Day\" writes:\n>\n> > Currently, all of the examples for \"man git-rebase\" show rebasing from\n> > a branch that has had no further development, which might mislead\n> > readers into thinking that that is a necessary condition for rebasing,\n> > so tweak the examples to show further development on such a branch to\n> > clarify that.\n>\n> We state the status-quo in present tense to start problem\n> description, so \"Currently\" is a noise word you can and should omit.\n>\n> As I already said, at least one example that rebases a branch that\n> was forked from the midpoint of another branch, so the problem\n> description is already false. If we apply this patch, do we lose\n> all examples that rebase a branch that purely builds on top of the\n> tip of another branch? That would also mislead readers into\n> thinking that you need to advance the base branch before you can\n> rebase the forked branch ;-).\n\n i stand corrected. carry on. :-)\n\nrday\n\n-- \n\n========================================================================\nRobert P. J. Day Ottawa, Ontario, CANADA\n http://crashcourse.ca/dokuwiki\n\nTwitter: http://twitter.com/rpjday\nLinkedIn: http://ca.linkedin.com/in/rpjday\n========================================================================\n"}]}