Previously, specifying
git log --pretty=format:'%H %s' --decorate
would calculate decorations, but not show them. You can now
do:
git log --pretty=format:'%H (%d) %s' --decorate
to see them.
Signed-off-by: Jeff King <peff@xxxxxxxx>
---
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.
Is it too loose? Perhaps the enclosing parentheses should be
automatic, so that %d expands to nothing if there is no decoration,
or the whole thing otherwise. Right now you are stuck with empty ()
if there is no decoration. Alternatively, we could support some kind
of conditional expansion in the formatting language (but I don't
know how crazy we want to get wit new formatting features).
- should this turn on --decorate automatically? If you use '%d'
without --decorate, you will just get no decorations. I think that
makes sense, though, since that opens room for specifying other
types of decorations (e.g., there could be a --decorate-tags that
only looks at tags).
Documentation/pretty-formats.txt | 1 +
pretty.c | 15 +++++++++++++++
2 files changed, 16 insertions(+), 0 deletions(-)
diff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt
index c11d495..55a5954 100644
--- a/Documentation/pretty-formats.txt
+++ b/Documentation/pretty-formats.txt
@@ -116,6 +116,7 @@ The placeholders are:
- '%cr': committer date, relative
- '%ct': committer date, UNIX timestamp
- '%ci': committer date, ISO 8601 format
+- '%d': decoration (if you specified --decorate)
- '%e': encoding
- '%s': subject
- '%b': body
diff --git a/pretty.c b/pretty.c
index 33ef34a..00f19e1 100644
--- a/pretty.c
+++ b/pretty.c
@@ -519,6 +519,21 @@ static size_t format_commit_item(struct strbuf *sb, const char *placeholder,
return 3;
} else
return 0;
+ case 'd':
+ {
+ struct name_decoration *d;
+ const char *prefix = "";
+ d = lookup_decoration(&name_decoration,
+ &commit->object);
+ while (d) {
+ strbuf_addstr(sb, prefix);
+ prefix = ", ";
+ strbuf_addstr(sb, d->name);
+ d = d->next;
+ }
+ }
+ return 1;
+
}
/* these depend on the commit */
--
1.6.0.90.g00a5c.dirty
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
