From: Ilya Tumaykin <itumaykin@gmail.com>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: git@vger.kernel.org
Subject: Re: Bug: "git log --format='format:%+s%+b'" doesn't insert newline before body
Date: Mon, 25 Jul 2016 10:51:13 +0300 [thread overview]
Message-ID: <1693749.WYZjdcdZj5@photon> (raw)
In-Reply-To: <alpine.DEB.2.20.1607250926320.14111@virtualbox>
On Monday 25 July 2016 09:33:00 Johannes Schindelin wrote:
> Hi Ilya,
>
> On Sun, 24 Jul 2016, Ilya Tumaykin wrote:
> > Steps to reproduce:
> > $ git init
> > $ >123
> > $ git add 123
> > $ git commit -v -m 'This is subject' -m 'And this is body'
> > $ git --no-pager log -1 --format='format:%+s%+b'
> >
> > Actual results:
> > ```
> >
> > This is subject
> > And this is body
> > ```
> >
> > Expected results:
> > ```
> >
> > This is subject
> >
> > And this is body
> > ```
>
> The empty line between commit subject and body is neither part of the
> subject nor of the body. That means that the above-mentioned expectation
> was incorrect.
According to 'git-log' man page '%+b' should insert linefeed "immediately
before the expansion if and only if the placeholder expands to a non-empty
string." Here "%b" expands to a non-empty string, thus I expect a linefeed
before it. Or am I misinterpreting man page somehow?
> Unless you somehow allow empty commit messages (Git does not, unless you
> play games with low-level commands), the second '+' is unnecessarily
> conditional. Therefore "%s%n%+b" *might* do what you intended (I would not
> know, because that information was missing from the report).
I want to display '%B', but add colours to '%s'. Thus I have to separately
include '%s' and '%b', and not just '%B'. I was using '%+s%n%+b' with extra
colour formatting as a workaround for some time now.
> Ciao,
> Johannes
--
Best regards.
Ilya Tumaykin.
next prev parent reply other threads:[~2016-07-25 7:51 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-24 13:46 Bug: "git log --format='format:%+s%+b'" doesn't insert newline before body Ilya Tumaykin
2016-07-25 6:55 ` Bug: Beat Bolli
2016-07-25 7:33 ` Bug: "git log --format='format:%+s%+b'" doesn't insert newline before body Johannes Schindelin
2016-07-25 7:51 ` Ilya Tumaykin [this message]
2016-07-25 8:06 ` Johannes Schindelin
2016-07-25 10:49 ` Ilya Tumaykin
2016-07-25 17:28 ` Jakub Narębski
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=1693749.WYZjdcdZj5@photon \
--to=itumaykin@gmail.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=git@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.