Re: [PATCH v12 11/11] Documentation: add documentation for 'git interpret-trailers'

From: Jakub Narębski <jnareb@gmail.com>
Date: Jun 30, 2014, 11:57 UTC
Message-ID: <53B150B4.1030303@gmail.com>
In-Reply-To: <20140525053223.5329.28002.chriscool@tuxfamily.org>

Christian Couder wrote:

Show 13 quoted lines

> +------------
> +
> +* Configure a 'sign' trailer with a command to automatically add a
> +  'Signed-off-by: ' with the author information only if there is no
> +  'Signed-off-by: ' already, and show how it works:
> ++
> +------------
> +$ git config trailer.sign.key "Signed-off-by: "
> +$ git config trailer.sign.ifmissing add
> +$ git config trailer.sign.ifexists doNothing
> +$ git config trailer.sign.command 'echo "$(git config user.name) <$(git config user.email)>"'
> +$ git interpret-trailers <<EOF
> +> EOF

How to configure git-interpret-trailers command so that it follow
current rules for DCO:
* Signed-off-by: is always at bottom; we can have
   signoff+signoff+ack+signoff
* Signed-off-by: can repeat itself with the same author;
   this denotes steps in coming up with current version of the patch.
* but we shouldn't repeat the same signoff one after another

So we want to allow this:

   Signed-off-by: A U Thor <author@example.com>
   Signed-off-by: Joe R. Hacker <joe@hacker.com>
   Acked-by: D E Veloper <developer@example.com>
   Signed-off-by: C O Mitter <committer@example.com>

but prevent this

   Signed-off-by: C O Mitter <committer@example.com>
   Signed-off-by: C O Mitter <committer@example.com>

IIRC

-- 
Jakub Narębski