On Mon, 4 Feb 2008, David Tweed wrote:
In principle this is a really trivial patch, but I'm being cautious because I
don't like the fact that I don't know (and AFAICS can't reliably check) that
the files being deleting are definitely dead. An alternative would
be to make prune just print out that the suspicious packs are
there and let the user delete them manually. (My itch is that once
a write-failure pack gets created, nothing in git operations tells the
user that a generally multimegabyte file hidden in .git occupying space.)
The lifelessness of a temporary pack is the same as for loose objects,
hence the same rule should apply in both cases. Just asking the user to
delete them manually isn't too nice either. A prune operation is
already said to be dangerous and should be performed only when no other
activities are occurring in the same repository. That should cover the
case of dead temporary pack files just as well.
Nicolas