Re: [PATCH v2] Add configuration variable for sign-off to format-patch

From: Jeff King <peff@peff.net>
Date: Apr 7, 2009, 07:23 UTC
Message-ID: <20090407072327.GA5239@coredump.intra.peff.net>
In-Reply-To: <49D9E040.40007@op5.se>

On Mon, Apr 06, 2009 at 12:58:08PM +0200, Andreas Ericsson wrote:

> How about:
> "Some projects (notably the Linux kernel and git itself) put special
> meaning in a 'Signed-off-by' line while other's dont. Please refer
> to your project's documentation for appropriate behaviour."

I think that is fine, with some typo fixups:

  s/other's/others/
  s/dont/don't/

And of course being American, I would spell it behavior. :)

Show 5 quoted lines

> Optionally with the following amendment:
>
> "In general, you should refrain from signing off on a patch containing
> material that you're not sure can be legally spread under the project's
> license."

I don't think that makes sense. That is what signing-off is about for the kernel and for git, but I don't think there is any reason another project might not want to use the convenient signoff options to mean something totally unrelated (like we use Acked-by).

-Peff