git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: "Jason Pyeron" <jpyeron@pdinc.us>
Cc: <git@vger.kernel.org>
Subject: Re: add a blank line when a commit has no parent in log output?
Date: Thu, 14 Jan 2021 17:12:05 -0800	[thread overview]
Message-ID: <xmqq8s8vvw9m.fsf@gitster.c.googlers.com> (raw)
In-Reply-To: <191201d6eaa3$4b585fa0$e2091ee0$@pdinc.us> (Jason Pyeron's message of "Thu, 14 Jan 2021 13:30:11 -0500")

"Jason Pyeron" <jpyeron@pdinc.us> writes:

> Is there a way to have it look like:
>
> | | | *  5505e019c2 2014-07-09 initial xxxxxx@xxxx
> | | |
> | | | *  3e658f4085 2019-09-10 (wiki/wip-citest, origin/wip-citest) Added defau
> | | | *  ad148aafe6 2019-09-10 Added default CI/CD Jenkinsfile (from f7daf088)
>
> Or 
>
> | | | #  5505e019c2 2014-07-09 initial xxxxxx@xxxx
> | | | *  3e658f4085 2019-09-10 (wiki/wip-citest, origin/wip-citest) Added defau
> | | | *  ad148aafe6 2019-09-10 Added default CI/CD Jenkinsfile (from f7daf088)

This latter variant won't work.  Imagine we are showing --left-right
for example.  Which side does '#' belong to?

The former is not so great in that it wastes a line, and the break
won't be as noticeable when --graph is *not* used with --oneline.

It would be great to show it more like this:

 | | |   * 5505e019c2 2014-07-09 initial xxxxxx@xxxx
 | | | *  3e658f4085 2019-09-10 (wiki/wip-citest, origin/wip-citest) Added defau
 | | | *  ad148aafe6 2019-09-10 Added default CI/CD Jenkinsfile (from f7daf088)

The point being that by shifting the column for the commit to the
right, it shows that 5505 is not a child of 3e65 (and 3e65 is the
tip of its lineage), and its parents do not appear in the displayed
history.  In the real life, the independent 'root' may be connected
to the main history somehow, so you may see a graph like this:

 | | *  12345678 2021-01-14 merge xxxxx@xxxx into the history  
 | | |\
 | | | \
 | | *  \  23456789 2021-01-12 merge citest into the main history
 | | |\  * 5505e019c2 2014-07-09 initial xxxxxx@xxxx
 | | | *  3e658f4085 2019-09-10 (wiki/wip-citest, origin/wip-citest) Added defau
 | | | *  ad148aafe6 2019-09-10 Added default CI/CD Jenkinsfile (from f7daf088)


Hmm?

      parent reply	other threads:[~2021-01-15  1:13 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-14 18:30 add a blank line when a commit has no parent in log output? Jason Pyeron
2021-01-14 19:29 ` Philippe Blain
2021-01-14 20:44   ` Jason Pyeron
2021-01-17 11:03     ` [PATCH 0/2] Option to modify revision mark for root commits Kyle Marek
2021-01-17 11:03       ` [PATCH 1/2] revision: Denote root commits with '#' Kyle Marek
2021-01-17 21:10         ` Junio C Hamano
2021-01-18  7:56           ` Kyle Marek
2021-01-18 19:15             ` Junio C Hamano
2021-01-18 20:33               ` Junio C Hamano
2021-01-19  7:43                 ` Kyle Marek
2021-01-19 22:10                   ` Junio C Hamano
2021-01-20  3:25                     ` Kyle Marek
2021-01-20  6:47                       ` Junio C Hamano
2021-01-20 15:11                         ` Jason Pyeron
2021-01-20 21:52                           ` Junio C Hamano
2021-01-20 23:01                             ` Jason Pyeron
2021-01-23 18:07                               ` Junio C Hamano
2021-01-23 23:02                                 ` Jason Pyeron
2021-01-23 23:45                                   ` Junio C Hamano
2021-01-24  0:02                                     ` Jason Pyeron
2021-01-25  7:00                                       ` Junio C Hamano
2021-01-17 22:44         ` Junio C Hamano
2021-01-17 11:03       ` [PATCH 2/2] revision: implement --show-linear-break for --graph Kyle Marek
2021-01-17 22:56         ` Junio C Hamano
2021-01-18  2:09           ` Junio C Hamano
2021-01-18  7:56             ` Kyle Marek
2021-01-18 21:01               ` Junio C Hamano
2021-01-19  7:44                 ` Kyle Marek
2021-01-15  1:12 ` Junio C Hamano [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqq8s8vvw9m.fsf@gitster.c.googlers.com \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=jpyeron@pdinc.us \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).