From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp1.linuxfoundation.org (smtp1.linux-foundation.org [172.17.192.35]) by mail.linuxfoundation.org (Postfix) with ESMTPS id 38379982 for ; Fri, 10 Jul 2015 21:07:54 +0000 (UTC) Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) by smtp1.linuxfoundation.org (Postfix) with ESMTPS id 8A25A1B2 for ; Fri, 10 Jul 2015 21:07:53 +0000 (UTC) Date: Fri, 10 Jul 2015 14:07:50 -0700 From: josh@joshtriplett.org To: David Woodhouse Message-ID: <20150710210750.GF10533@cloud> References: <20150708114011.3a1f1861@noble> <2879113.fraeuJIr2M@avalon> <20150709193718.GD9169@vmdeb7> <1436481109.3324.219.camel@infradead.org> <20150710003559.GT11162@sirena.org.uk> <20150710020706.GH111846@vmdeb7> <20150710155144.34dde697@gandalf.local.home> <1436558412.24408.35.camel@infradead.org> <20150710205606.GC10533@cloud> <1436562223.24408.55.camel@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1436562223.24408.55.camel@infradead.org> Cc: ksummit-discuss@lists.linuxfoundation.org, Jason Cooper Subject: Re: [Ksummit-discuss] [CORE TOPIC] Recruitment (Reviewers, Testers, Maintainers, Hobbyists) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, Jul 10, 2015 at 10:03:43PM +0100, David Woodhouse wrote: > On Fri, 2015-07-10 at 13:56 -0700, josh@joshtriplett.org wrote: > > On Fri, Jul 10, 2015 at 09:00:12PM +0100, David Woodhouse wrote: > > > And then I can *read* them before sending them, which is good practice > > > anyway. Am I the only person who often finds a final minor nit with > > > their own patch, in that final read-through just before hitting 'send' > > > on an email? > > > > Nope, I do that too. (Though then I have to resist the temptation to > > just fix it in the mailer rather than fixing the original and > > regenerating.) > > Oh, I *always* just fix it in the mailer, even when the numbers in the > hunk headers need to be fixed up :) I've occasionally fixed up actual typos in the patch content, though never changing the number of lines. The danger there is what happens when you need to prepare v(N+1). If you're going to need to change it in git anyway, you might as well go ahead and do so and regenerate the series. I just wish I had a more satisfactory method of associating a cover letter with a series *in git*, such that format-patch could emit a non-placeholder cover letter. - Josh Triplett