[darcs-devel] [issue220] missing inventories file in repo for darcs project

[Juliusz Chroboczek]

> The first question I asked myself is why we are using cleanDate in
> PatchInfo.make_filename.  After all, the date comes from a PatchInfo,
> right? The impression I get is that the date format inside a patch has
> changed.  Around 2003, we used to write them in the human-readable way,
> and sometime later, we must have switched to a terse ISO format like
> 20031110125819.

Yes.

> There are only three uses of cleanDate.  The other two are for human
> interaction.  So rather than use cleanDate in this function, and
> rather than write a second cleanDate with incorrect behaviour, we
> simply slap a showIsoDateTime.readDate into PatchInfo.make_filename
> and put a comment saying that this is wrong, but we need it for old
> patches.

Yes.

                                        Juliusz