From: Dietmar Winkler <dietmarw@gmx.de>
To: Jeff King <peff@peff.net>
Cc: Junio C Hamano <gitster@pobox.com>,
Will Palmer <wmpalmer@gmail.com>,
git@vger.kernel.org
Subject: Re: Fwd: [PATCH 2/2] pretty.c: allow date formats in user format strings
Date: Fri, 11 Mar 2011 09:33:30 +0100 [thread overview]
Message-ID: <4D79DE5A.9060700@gmx.de> (raw)
In-Reply-To: <20110310223148.GD15828@sigill.intra.peff.net>
Den 10. mars 2011 23:31, skrev Jeff King:
> OK. Do you want me to throw away the %ad(format) patch for now, then, in
> favor of building it on top of a more sane syntax?
>
> I had originally planned to do %ad(format) for now, and then worry about
> syntax later. Since we already have a variety of of other placeholders
> with similar syntax (e.g., %w(), %C()). But I don't care too much either
> way; it is not a feature I personally wanted, so delay doesn't bother
> me. Dietmar (the original requestor) may feel differently, of course. :)
As much as I would like to have such a feature (and a documentation that
is in synch with the implementation ;) I don't heavily rely on it. I'm
happy to wait a bit longer in favour of a more complete and clean
implementation which also opens up for other %(longname:options) support.
--
Dietmar.
prev parent reply other threads:[~2011-03-11 8:33 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-03 9:30 [Bug] %[a|c]d placeholder does not respect --date= option in combination with git archive Dietmar Winkler
2011-03-03 15:10 ` Jeff King
2011-03-04 10:10 ` Dietmar Winkler
2011-03-05 19:50 ` Jeff King
2011-03-05 19:51 ` [PATCH 1/2] pretty.c: give format_person_part the whole placeholder Jeff King
2011-03-05 20:00 ` [PATCH 2/2] pretty.c: allow date formats in user format strings Jeff King
[not found] ` <AANLkTinH8zwX2sbd5bpk=x4R3zOAg3Dc92Fbspfdv03T@mail.gmail.com>
2011-03-06 21:54 ` Fwd: " Will Palmer
2011-03-07 16:17 ` Jeff King
2011-03-07 17:28 ` Will Palmer
2011-03-07 18:50 ` Will Palmer
2011-03-07 19:26 ` Jeff King
2011-03-08 8:29 ` Will Palmer
2011-03-09 21:06 ` Junio C Hamano
2011-03-10 22:31 ` Jeff King
2011-03-11 8:33 ` Dietmar Winkler [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=4D79DE5A.9060700@gmx.de \
--to=dietmarw@gmx.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=peff@peff.net \
--cc=wmpalmer@gmail.com \
/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.