login
Header Space

 
 

Re: [PATCH] rpc: fix garbage in printk in svc_tcp_accept()

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: J. Bruce Fields <bfields@...>
Cc: <linux-kernel@...>, Andrew Morton <akpm@...>, Wolfgang Walter <wolfgang.walter@...>, Neil Brown <neilb@...>, Trond Myklebust <trond@...>, <stable@...>
Date: Thursday, September 20, 2007 - 4:15 pm

On Thu, 20 Sep 2007, J. Bruce Fields wrote:
...

Bruce - you're losing authorship information.

Please don't do that.

Put a 

	From: Wolfgang Walter <wolfgang.walter@studentenwerk.mhn.de>

at the top of the email, so that the author gets properly attributed. I 
don't know how many of these I've missed - in this case I just ended up 
googling for the commit message because the "Signed-off-by:" lines implied 
that you weren't the original author, and see the real patch authorship 
that way.

But it's *not* the case that the first sign-off is always the author, and 
I don't want to need to google for each patch I get, so please make it a 
habit to make sure that there's proper attribution for the patches.

Some people (read: Andrew) go so far as to *always* do it (ie Andrew puts 
a "From: Andrew Morton .." at the heads of the emails he sends out), and I 
appreciate it. That way, even if the email then gets forwarded by somebody 
else (because Andrew sent it to some submaintainer instead of me), the 
original authorship doesn't get lost when somebody else then forwards the 
email.

So that "always explicitly say who the author is" is a good idea, but when 
forwarding somebody elses patch, it's *more* than a good idea - at that 
point it's absolutely required for proper attribution.

		Linus
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] rpc: fix garbage in printk in svc_tcp_accept(), J. Bruce Fields, (Thu Sep 20, 3:51 pm)
Re: [PATCH] rpc: fix garbage in printk in svc_tcp_accept(), Linus Torvalds, (Thu Sep 20, 4:15 pm)
Re: [PATCH] rpc: fix garbage in printk in svc_tcp_accept(), J. Bruce Fields, (Thu Sep 20, 4:21 pm)
speck-geostationary