On Wed, Jul 1, 2015 at 1:49 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Dave Borowitz <dborowitz@xxxxxxxxxx> writes: > >>> I am moderately negative about this; wouldn't it make the end result >>> cleaner to fix the implementation? >> >> I'm not sure I understand your suggestion. Are you saying, you would >> prefer to make LFs optional in the push cert, for consistency with LFs >> being optional elsewhere? > > Absolutely. It is not "make" it optional, but "even though it is > optional, the receiver has not been following the spec, and it is > not too late to fix it". This is madness. For all the reasons Dave points out later in the thread. You can't store and make sense of the push cert without the LF record delimiters. push cert format is like commit or tag format. You need those LFs. We can't just go declare them optional because of the way pkt-line read function is implemented in git-core. -- 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