Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: >> +for name in $names >> +do >> + rm -f "$PACKDIR/old-pack-$name.idx" >> + rm -f "$PACKDIR/old-pack-$name.pack" >> +done >> + >> +# End of pack replacement. > > Is this comment really needed here? Is that hunk still needed? I think the removal part is already there all the way down to maint. Other than that, I think your comments are all sane. -- 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
