Junio C Hamano <junkio@xxxxxxx> wrote: > "Shawn O. Pearce" <spearce@xxxxxxxxxxx> writes: > > > My recent patch "Lazily open pack index files on demand" caused a > > regression in the case of parsing abbreviated SHA-1 object names. > > Git was unable to translate the abbreviated name into the full name > > if the object was packed, as the pack .idx files were not opened > > before being accessed. > > Thanks. As long as we catch it before it goes to 'master', it > is not a regression ;-) Good point. Next time I won't call it a regression. But to me, anything that hits 'next' that breaks Git this badly is a regression. Why? Because I run my production repositories off next, that's why. Of course I do this to exercise next more fully... to prevent this sort of stuff from getting to master. ;-) For what its worth, I'm going through the code again and auditing our use of the index related variables in packed_git that are now lazily loaded. I'm not seeing anything that is critical. I do have two minor patches queued up, but they are just to make things look prettier. Will send soon. -- Shawn. - 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
