Felipe Contreras <felipe.contreras@xxxxxxxxx> writes:
> Junio C Hamano wrote:
>> Felipe Contreras <felipe.contreras@xxxxxxxxx> writes:
>>
>> >> But if you are arguing that when you write "Signed-off-by:" your
>> >> sign-off can mean something other than what DCO says it means,
>> >
>> > The DCO has clause (d), which clearly states the developer must agree
>> > that a record of his/her contribution is maintained indefinitely (and
>> > that includes his/her sign off).
>>
>> Yes. Are you saying that you are OK with (a)-(c) but not (d)?
>
> I'm saying if the author of the patch states "I don't agree with a
> record of my contribution being maintained indefinitely with my sign
> off", then clause (d) isn't met.
Yeah, but then why does such an author add Signed-off-by: trailer to
begin with? Here is what Documentation/SubmittingPatches tells the
authors:
=== Certify your work by adding your `Signed-off-by` trailer
To improve tracking of who did what, we ask you to certify that you
wrote the patch or have the right to pass it on under the same license
as ours, by "signing off" your patch. Without sign-off, we cannot
accept your patches.
If you can certify the below D-C-O:
[[dco]]
.Developer's Certificate of Origin 1.1
____
By making a contribution to this project, I certify that:
a. ...; or
b. ...; 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.
d. I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
____
you add a "Signed-off-by" trailer to your commit, that looks like
this:
So, "by making a contribution", the author who added a Signed-off-by
trailer is certifying that (a|b|c)&d is true.
Perhaps we can tighten the language to say "If (and only if) you can
certify" and that may reduce confusion?
