On Thu, 25 Jan 2007, Junio C Hamano wrote:
But I think "only packing tags" is a reasonable default
Sure, I agree. I just think that it had the unintended side effect when
the code then didn't repack stuff that had already been packed earlier.
Which is why my fix doesn't actually change the fact that we only pack
tags by default, but just fixes the thing that caused us to then totally
drop previously packed non-tags.
So now the default isn't really "pack only tags", but "pack only tags *or*
things that were already packed".
The reason, of course, is that if we don't pack stuff that used to be
packed, it will just be dropped entirely. Oops.
Linus