lkml.org 
[lkml]   [2006]   [Jun]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: reviewing Ubuntu kernel patches
On Fri, Jun 16, 2006 at 10:03:34AM -0400, Theodore Tso wrote:
> On Thu, Jun 15, 2006 at 10:39:49PM -0700, Randy Dunlap wrote:
> > Certainly a good question IMO. If Andrew or Linus knows whether
> > I need to add my Signed-off-by, I'll be glad to listen.
> > (That's not a general call for opinions.)
>
> If you're submitting the patch, then surely you need to add your
> Signed-off-by:, since you're asserting either (b) or (c):
>
> (b) The contribution is based upon previous work that, to the best
> of my knowledge, is covered under an appropriate open source
> license and I have the right under that license to submit that
> work with modifications, whether created in whole or in part
> by me, under the same open source license (unless I am
> permitted to submit under a different license), as indicated
> in the file; or
>
> (c) The contribution was provided directly to me by some other
> person who certified (a), (b) or (c) and I have not modified
> it.
>
> We've gotten into the habit of assuming the Signed-off-by: also has
> the meaning of "I vouch for it from technical point of view", but
> really, that's presumably true since otherwise you wouldn't have
> e-mailed it to Andrew or Linus. The original meaning of the
> Signed-off-by: is in the Developer's Certificate of Origin
> statement...

Re-reading this text, I'm getting some doubts whether the way Andrew
handles this when merging patches is correct:

When Andrew merges some small fix into an existing patch in -mm
(e.g. "make a function static" or "fix a compile error"), he adds the
Signed-off-by line of the small fix to the Signed-off-by lines of the
patch.

But the submitter of the fix does not necessarly have checked the
legal or technical status of the rest of the patch.

As an example, my only contribution to commit
6b3934ef52712ece50605dfc72e55d00c580831a was making signal_cachep
static, and I do refuse any legal or technical responsibility for
the rest of the patch (this shouldn't imply the rest of the patch was
bad - it's simply not me who can is responsible for it).

> - Ted

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-06-21 19:41    [W:0.068 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site