From: Junio C Hamano Date: Wed, 30 Jan 2013 17:37:54 GMT Subject: Re: [PATCH v3 00/11] unify appending of sob Message-ID: <7vhalylggd.fsf@alter.siamese.dyndns.org> In-Reply-To: <7v7gmxzwkj.fsf@alter.siamese.dyndns.org> Junio C Hamano writes: > Jonathan Nieder writes: > >> Brandon Casey wrote: >> >>> Round 3. >> >> Thanks for a pleasant read. My only remaining observations are >> cosmetic, except for a portability question in Duy's test script, a >> small behavior change when the commit message ends with an >> RFC2822-style header with no trailing newline and the possibility of >> tightening the pattern in sequencer.c to match the strictness of >> format-patch (which could easily wait for a later patch). > > Thanks for a quick review. I agree that this series is getting very > close with your help. Unless Brandon and/or Jonathan wants to have another chance to excise warts from the recorded history by rerolling the entire series one more time, I think what we have queued is in a good enough shape to merge to 'next' and any further improvement and fix can be done incrementally. OK? Or "stop, I want to reroll"? I'll wait for a day or two.