From: Junio C Hamano Date: Wed, 22 Feb 2012 20:50:51 GMT Subject: Re: [PATCH 2/2] bundle: use a strbuf to scan the log for boundary commits Message-ID: <7vhayia8yc.fsf@alter.siamese.dyndns.org> In-Reply-To: <87hayivcmm.fsf@thomas.inf.ethz.ch> Thomas Rast writes: > In any case the --pretty=oneline is very deliberate, as we can see from > the commit below. It just doesn't give a reason :-) Thanks for digging and I hope everybody on this list who would ever send a patch learns the importance of describing _why_ in the log from this lesson. I'll amend your 2/2 with a note that keeping the subject material is not justified and we may want to remove (or at least reduce) it in the future.