threads / bug / 40248

Git book figure bug

Subject: Git book figure bug

## tl;dr

3 messages between Sep 2, 2015 and Sep 2, 2015.

replies: 2people: 3as markdown or json

Jesper Jørgensen· Sep 2, 2015, 07:45 UTC · lore
Hi,

I am learning how git works, and is using the excellent book on the official site. I believe I have located a mistake in one of the images. It is about Figure 3.17 on this page : http://git-scm.com/book/en/v2/Git-Branching-Basic-Branching-and-Merging

As far as I understand it, the right most snapshot called C5 should have been named C6. The texts describes a new snapshot being made from the merge of the 2 branches. Having 2 snapshots both named C5 does not seem like it can be possible.

Best regards JESPER JØRGENSEN, Prevas A/S Software developer, Aarhus Hedeager 1, DK-8200 Aarhus N Phone +45 87438070 Mobile +45 40282198 jesper.jorgensen@prevas.dk www.prevas.dk

 
Chris Packham· Sep 2, 2015, 08:36 UTC · re: Jesper Jørgensen · lore

Re: Git book figure bug

On Wed, Sep 2, 2015 at 7:45 PM, Jesper Jørgensen <Jesper.Jorgensen@prevas.dk> wrote:

Show 10 quoted lines
> Hi,
>
> I am learning how git works, and is using the excellent book on the official site.
> I believe I have located a mistake in one of the images.
> It is  about Figure 3.17 on this page : http://git-scm.com/book/en/v2/Git-Branching-Basic-Branching-and-Merging
>
> As far as I understand it, the right most snapshot called C5 should have been named C6.
> The texts describes a new snapshot being made from the merge of the 2 branches.
> Having 2 snapshots both named C5 does not seem like it can be possible.
>

Indeed you are correct. Looks like this was fixed a while ago https://github.com/progit/progit2/commit/3491527b33251fcacd8d2964bf7fb5e0f1cfe455 but hasn't made it into the "live" website.

Johannes Schindelin· Sep 2, 2015, 08:39 UTC · re: Jesper Jørgensen · lore

Re: Git book figure bug

Hi Jesper,
On 2015-09-02 09:45, Jesper Jørgensen wrote:
Show 10 quoted lines
> I am learning how git works, and is using the excellent book on the
> official site.
> I believe I have located a mistake in one of the images.
> It is  about Figure 3.17 on this page :
> http://git-scm.com/book/en/v2/Git-Branching-Basic-Branching-and-Merging
> 
> As far as I understand it, the right most snapshot called C5 should
> have been named C6.
> The texts describes a new snapshot being made from the merge of the 2 branches.
> Having 2 snapshots both named C5 does not seem like it can be possible.
Your hunch is correct, and it should be C6, that's why it was changed:
https://github.com/progit/progit2/blob/master/book/03-git-branching/images/basic-merging-2.png
I do not know why it is still displayed incorrectly on https://git-scm.com/, but your best bet would be to report this as a bug: https://github.com/git/git-scm.com/issues
But make sure first that there is not another ticket opened for the same bug!

Ciao, Johannes

← back to recent threads