From: Teemu Likonen <tlikonen@iki.fi>
To: Jeff King <peff@peff.net>
Cc: MichaelTiloDressel@t-online.de,
Junio C Hamano <gitster@pobox.com>,
git@vger.kernel.org
Subject: "log --pretty=format:" language
Date: Thu, 21 Aug 2008 08:02:23 +0300 [thread overview]
Message-ID: <20080821050223.GA3295@mithlond.arda.local> (raw)
In-Reply-To: <20080820180034.GB32005@sigill.intra.peff.net>
Jeff King wrote (2008-08-20 14:00 -0400):
> There is a lot of room for discussion here.
>
> For example:
>
> - what should %d show? Right now it shows each decoration, split by
> commas. It doesn't show the enclosing parentheses automatically.
>
> Is this too strict? Should there be some way of pulling out
> individual decorations from the list, or specifying a different
> delimiter? If so, probably that should be part of a general
> improvement in the format expansion macro language.
If such "general improvement" takes place I'd like to point out (most
likely old news, but anyway) that %b can't be indented in practical
sense. For example, the command
git log --pretty=format:%x09%s%n%n%x09%b -1 3a634dc
prints this:
Add hints to revert documentation about other ways to undo changes
Based on its name, people may read the 'git revert' documentation when
they want to undo local changes, especially people who have used other
SCM's. 'git revert' may not be what they had in mind, but git
provides several other ways to undo changes to files. We can help
them by pointing them towards the git commands that do what they might
want to do.
[...]
next prev parent reply other threads:[~2008-08-21 5:04 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-20 12:25 [BUG?] git log does not decorate when custom format is used MichaelTiloDressel
2008-08-20 17:53 ` Jeff King
2008-08-20 17:55 ` [PATCH 1/2] decorate: allow const objects to be decorated Jeff King
2008-08-20 18:00 ` [PATCH 2/2] allow '%d' pretty format specifier to show decoration Jeff King
2008-08-20 18:32 ` Junio C Hamano
2008-08-20 18:43 ` Johannes Schindelin
2008-08-20 19:01 ` Junio C Hamano
2008-08-20 19:53 ` Johannes Schindelin
2008-09-03 19:36 ` Jeff King
2008-08-20 19:51 ` Michael Dressel
2008-08-20 20:21 ` Junio C Hamano
2008-08-20 21:05 ` Michael Dressel
2008-08-21 5:02 ` Teemu Likonen [this message]
2008-08-24 18:30 ` "log --pretty=format:" language Jakub Narebski
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=20080821050223.GA3295@mithlond.arda.local \
--to=tlikonen@iki.fi \
--cc=MichaelTiloDressel@t-online.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=peff@peff.net \
/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).