From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755737AbaIQPwF (ORCPT ); Wed, 17 Sep 2014 11:52:05 -0400 Received: from imap.thunk.org ([74.207.234.97]:33923 "EHLO imap.thunk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755626AbaIQPuP (ORCPT ); Wed, 17 Sep 2014 11:50:15 -0400 Date: Wed, 17 Sep 2014 11:49:18 -0400 From: "Theodore Ts'o" To: Milosz Tanski Cc: Benjamin LaHaise , Dave Chinner , Jeff Moyer , LKML , Christoph Hellwig , "linux-fsdevel@vger.kernel.org" , linux-aio@kvack.org, Mel Gorman , Volker Lendecke , Tejun Heo Subject: Re: [PATCH 7/7] check for O_NONBLOCK in all read_iter instances Message-ID: <20140917154918.GB12190@thunk.org> Mail-Followup-To: Theodore Ts'o , Milosz Tanski , Benjamin LaHaise , Dave Chinner , Jeff Moyer , LKML , Christoph Hellwig , "linux-fsdevel@vger.kernel.org" , linux-aio@kvack.org, Mel Gorman , Volker Lendecke , Tejun Heo References: <20140916214254.GK4322@dastard> <20140917122448.GR24821@kvack.org> <20140917134702.GB10720@thunk.org> <20140917135611.GT24821@kvack.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: tytso@thunk.org X-SA-Exim-Scanned: No (on imap.thunk.org); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 17, 2014 at 11:33:35AM -0400, Milosz Tanski wrote: > > My workflow has been to use git format-patch (1.7.9.5 was shipped with > my distro), edit the cover letter then use and mutt to send the > generated emails. Before that I used git apply to import the patches > that Christoph sent me. I thought about it too, but hand editing the > email generated by format-patch to essentially having me take credit > for this sounded like a shady thing to do. The updated version of > git's (2.1.0) format-patch doesn't change the from email address field > either. That's because it gets handled in git send-email. The resulting e-mail gets sent like this: From: Subsystem Maintainer To: LKML list Subject: [PATCH] ext4: foobie blart From: Original Author Long commit description which gets placed as the third line of the commit description, with the subject line as the first line of the commit description, and the 2nd line being blank. And then git am handles this correctly, attributing the patch authorship to Original Author, with the git committer set to the Subsystem Maintainer who ran the "git am" command. > The submitting patches document doesn't really describe what to do if > you take patches / collaborate with somebody else or how to credit the > original author. It only deals with the case of a subsystem > maintainers editing the submitters code to fix it up. The short version is, use a non-prehistoric version of git, and use "git format-patch" and "git send-email", and be happy. :-) It's not that hard to build your own version of git, if you are forced to use some enterprise/LTS/Debian stable distro that can't be bothered to update git for you. Cheers, - Ted