Re: [PATCH 0/2] Use %as and %cs as pretty format flags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 28, 2008 at 06:09:37AM -0500, Nathan Panike wrote:

> The patch series below allows git to use %as and %cs as flags for
> pretty-printing the format of a date.

Your description leaves a little to be desired (here and in the patches
themselves). I had to read the patch to figure out that these are
formatting specifiers for the date format "short".

That being said, I think this is probably reasonable just for the sake
of completeness (and I doubt we are wasting a useful formatting combo,
since %a* is likely to remain dedicated to author information). I wonder
if there should be "%al" for "local".

However, it makes me wonder even more if '%ad' should simply respect the
--date= parameter (this wouldn't allow you to mix and match dates in a
single format, but I don't think that is what is desired). Or whether we
should have some syntax for "%ad(short)" or something, where the
argument would be handed off to the date format parser. But that is
probably overengineering.

-Peff

PS Junio, I think this will make your "git one" alias much shorter. :)
--
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux