On Tue, 20 Mar 2007, Andy Parkins wrote: > On Tuesday 2007 March 20 08:49, Junio C Hamano wrote: > > > >> noticed, because almost nobody uses it. > > > > > > I'm not sure that's going to be true for long - the 1.5.0 release notes > > > recommended setting it (assuming you didn't need backward compatibility) > > > - which is exactly what I (and I'm sure others) did. > > > > Well, it is fixed in 'master' to be in -rc2, and that validation > > does not exist in 'maint', so no harm is done. > > It wasn't the presence of the bug I was highlighting - it was the idea that > nobody uses that option. Yeah, I'm actually happy that people seem to be using it, and it looks like the bug really was totally harmless apart from triggering a bogus error (ie it would never have corrupted anything, it just triggered an error that it shouldn't have). And I think the only thing that could ever trigger it was really that special case of a zero-sized blob (or maybe an empty tree, but git doesn't generate those natively unless you do magic stuff by hand). Linus - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html