Part 3, wherein I regroup, and once again present an option where Signed-off-by is optional. (Or ... required :)
https://gitlab.freedesktop.org/mesa/mesa/merge_requests/31 I turned it into 3 patches. > 21f1070b6ef docs: Add developer-certificate-of-origin.txt Adds the DCO 1.1 as a separate text file. > c6213abf55d docs: Document the optional usage of Signed-off-by 2/3 points at the DCO 1.1 text and says that if you add Signed-off-by, then your contribution follows the DCO 1.1 guidelines. But, the usage of Signed-off-by would still be optional. > f510fb75cd2 docs: Document and *require* usage of Signed-off-by Make Signed-off-by required. So, bring your pitchforks, torches, NACKs and other implements of discussion to MR #31. :) -Jordan On 2018-11-27 23:20:22, Jordan Justen wrote: > This adds the "Developer's Certificate of Origin 1.1" from the Linux > kernel. It indicates that by using Signed-off-by you are certifying > that your patch meets the DCO 1.1 guidelines. > > It also changes Signed-off-by from being optional to being required. > > Signed-off-by: Jordan Justen <jordan.l.jus...@intel.com> > Cc: Matt Turner <matts...@gmail.com> > --- > docs/submittingpatches.html | 52 ++++++++++++++++++++++++++++++++++++- > 1 file changed, 51 insertions(+), 1 deletion(-) > > diff --git a/docs/submittingpatches.html b/docs/submittingpatches.html > index 9ae750d5a15..6d506b3691b 100644 > --- a/docs/submittingpatches.html > +++ b/docs/submittingpatches.html > @@ -20,6 +20,8 @@ > <ul> > <li><a href="#guidelines">Basic guidelines</a> > <li><a href="#formatting">Patch formatting</a> > +<li><a href="#signing">Patch Signing</a> (Signed-off-by, Developer's > + Certificate of Origin) > <li><a href="#testing">Testing Patches</a> > <li><a href="#mailing">Mailing Patches</a> > <li><a href="#reviewing">Reviewing Patches</a> > @@ -73,7 +75,9 @@ if needed. For example: > is necessary, and removing it causes no regressions in piglit on any > platform. > </pre> > -<li>A "Signed-off-by:" line is not required, but not discouraged either. > +<li>A "Signed-off-by:" line is <strong>required</strong>. The format > +and meaning of Signed-off-by is documented below in > +the <a href="#signing">patch signing</a> section. > <li>If a patch addresses a bugzilla issue, that should be noted in the > patch comment. For example: > <pre> > @@ -129,7 +133,53 @@ Please use common sense and do <strong>not</strong> > blindly add everyone. > </pre> > </ul> > > +<h2 id="signing"> > + Patch Signing (Signed-off-by, Developer's Certificate of Origin) > +</h2> > > +<p> > + As described in the <a href="#formatting">patch formatting</a> > + section, you must sign your patch by including Signed-off-by in the > + patch commit message. The Signed-off-by must include your real name > + and email address in this format: > +</p> > +<pre> > + Signed-off-by: Joe Hacker <jhac...@foo.com> > +</pre> > +<p> > + By adding Signed-of-by to your contributed patch, you certify that > + your contribution meets the guidelines of the Developer's > + Certificate of Origin: > +</p> > + > +<pre> > +Developer's Certificate of Origin 1.1 > +------------------------------------- > + > +By making a contribution to this project, I certify that: > + > + (a) The contribution was created in whole or in part by me and I > + have the right to submit it under the open source license > + indicated in the file; or > + > + (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. > + > + (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. > +</pre> > > <h2 id="testing">Testing Patches</h2> > > -- > 2.20.0.rc1 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
