Re: [PATCH 0/4] Tools: hv

[Greg KH <gregkh@linuxfoundation.org>]

On Wed, Sep 05, 2012 at 07:24:37PM +0000, KY Srinivasan wrote:
> > On Wed, Sep 05, 2012 at 10:01:59AM -0700, K. Y. Srinivasan wrote:
> > > This patchset was authored by Ben Hutchings <ben@decadent.org.uk>. Ben
> > > asked me to submit this patchset. This patchset cleans up the KVP daemon
> > > code and fixes some bugs.
> > >
> > >
> > > K. Y. Srinivasan (4):
> > >   tools: hv: Fix file handle leak
> > >   tools: hv: Fix exit() error code
> > >   tools: hv: Check for read/write errors
> > >   tools: hv: Parse /etc/os-release
> > 
> > If Ben wrote these, why are you showing that you wrote them?
> 
> Ben asked me to apply these patches on the tip (of my tree) and send the
> patches out and that is what I did. I recreated these patches after applying to
> my tree and that is what I have sent out. How could I have made the authorship
> more clear (other than noting so in the comment which I have done).

You could have actually _kept_ the proper authorship information so that
git shows who wrote the patch.  As your shortlog shows above, it looks
like you wrote the patches, not Ben, which is not acceptable.

As to how to actually do that, well, I'll leave that as an exercise for
the reader (hint, we document how to do this quite well...)

greg k-h