Code review comment for lp://staging/~jelmer/bzr-builder/revtime

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 06/10/2011 02:31 PM, Jelmer Vernooij wrote:
> Jelmer Vernooij has proposed merging lp:~jelmer/bzr-builder/revtime into lp:bzr-builder.
>
> Requested reviews:
> bzr-builder developers (bzr-builder-devs)
> Related bugs:
> Bug #793072 in bzr-builder: "Provide deb-version variables for revision commit time/date"
> https://bugs.launchpad.net/bzr-builder/+bug/793072
>
> For more details, see:
> https://code.launchpad.net/~jelmer/bzr-builder/revtime/+merge/64169
>
> Add {revtime}, {revdate} substitution variables.

It seems like dates are often spelled out as %Y-%m-%d rather than
%Y%m%d. I don't know where this would explicitly matter, but the former
is always obviously a date, while the latter can sometimes just be a
number, or something else.

Just a thought,
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk3yK/4ACgkQJdeBCYSNAAMivwCglVryPsEbWTimiGQL/MlbMkmH
llUAnjyjHa1dAvmfC4pCXKgheporT7NH
=i2bI
-----END PGP SIGNATURE-----