Modifying {isodate} to include seconds?

Giorgos Keramidas keramida at ceid.upatras.gr
Mon Mar 17 12:45:48 CDT 2008


On 2008-03-17 19:09, Giorgos Keramidas <keramida at ceid.upatras.gr> wrote:
> On 2008-03-17 12:02, Matt Mackall <mpm at selenic.com> wrote:
> >On Mon, 2008-03-17 at 17:49 +0200, Giorgos Keramidas wrote:
> >> I just noticed that {date|isodate} does not include seconds in our
> >> templater.
> >>
> >> $ pwd
> >> /home/keramida/hg/mercurial/crew/mercurial
> >> $ fgrep isodate *
> >> templatefilters.py:    "isodate": lambda x: util.datestr(x, '%Y-%m-%d %H:%M %1%2'),
> >> $
> >>
> >> Is it ok to modify this template to include the seconds?
> >
> > Ugh. That might break stuff. We may need to add an isodatesec.
>
> You are right.  We will definitely have to update the Mercurial test
> suite, if we change this (I'm doing it as I type this message).
>
> 'isodatesec' is an easy change, I guess.

Ok, a pair of patches for {isodatesec} has now been posted to
mercurial-devel :)



More information about the Mercurial-devel mailing list