* [PATCH 0/2] Fix Co-Developed-by @ 2018-12-14 17:35 Jorge Ramirez-Ortiz 2018-12-14 17:35 ` [PATCH 1/2] docs: fix Co-Developed-by docs Jorge Ramirez-Ortiz 2018-12-14 17:35 ` [PATCH 2/2] checkpatch: add Co-developed-by to signature tags Jorge Ramirez-Ortiz 0 siblings, 2 replies; 7+ messages in thread From: Jorge Ramirez-Ortiz @ 2018-12-14 17:35 UTC (permalink / raw) To: jorge.ramirez-ortiz, joe, gregkh, himanshujha199640 Cc: apw, akpm, linux-kernel, niklas.cassel, corbet, linux-doc The following set unifies the terminology for co-developed patches (losing the capital in Developed) and adds the rule to the checkpatch.pl script to stop warnings. Jorge Ramirez-Ortiz (2): docs: fix Co-Developed-by docs checkpatch: add Co-developed-by to signature tags Documentation/process/submitting-patches.rst | 4 ++-- scripts/checkpatch.pl | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) -- 2.7.4 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] docs: fix Co-Developed-by docs 2018-12-14 17:35 [PATCH 0/2] Fix Co-Developed-by Jorge Ramirez-Ortiz @ 2018-12-14 17:35 ` Jorge Ramirez-Ortiz 2018-12-14 18:12 ` Himanshu Jha 2018-12-14 17:35 ` [PATCH 2/2] checkpatch: add Co-developed-by to signature tags Jorge Ramirez-Ortiz 1 sibling, 1 reply; 7+ messages in thread From: Jorge Ramirez-Ortiz @ 2018-12-14 17:35 UTC (permalink / raw) To: jorge.ramirez-ortiz, joe, gregkh, himanshujha199640 Cc: apw, akpm, linux-kernel, niklas.cassel, corbet, linux-doc The accepted terminology will be Co-developed-by therefore losing the capital letter from now on. Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> --- Documentation/process/submitting-patches.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst index c091710..30dc00a 100644 --- a/Documentation/process/submitting-patches.rst +++ b/Documentation/process/submitting-patches.rst @@ -510,7 +510,7 @@ tracking your trees, and to people trying to troubleshoot bugs in your tree. -12) When to use Acked-by:, Cc:, and Co-Developed-by: +12) When to use Acked-by:, Cc:, and Co-developed-by: ------------------------------------------------------- The Signed-off-by: tag indicates that the signer was involved in the @@ -543,7 +543,7 @@ person it names - but it should indicate that this person was copied on the patch. This tag documents that potentially interested parties have been included in the discussion. -A Co-Developed-by: states that the patch was also created by another developer +A Co-developed-by: states that the patch was also created by another developer along with the original author. This is useful at times when multiple people work on a single patch. Note, this person also needs to have a Signed-off-by: line in the patch as well. -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] docs: fix Co-Developed-by docs 2018-12-14 17:35 ` [PATCH 1/2] docs: fix Co-Developed-by docs Jorge Ramirez-Ortiz @ 2018-12-14 18:12 ` Himanshu Jha 0 siblings, 0 replies; 7+ messages in thread From: Himanshu Jha @ 2018-12-14 18:12 UTC (permalink / raw) To: Jorge Ramirez-Ortiz, jic23, linux-iio Cc: joe, gregkh, apw, akpm, linux-kernel, niklas.cassel, corbet, linux-doc On Fri, Dec 14, 2018 at 06:35:27PM +0100, Jorge Ramirez-Ortiz wrote: > The accepted terminology will be Co-developed-by therefore losing the > capital letter from now on. > > Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> > --- Thanks. ACK! Finally, the discussion converged at somepoint. FYI, Cc Jonathan Cameron + IIO list. > Documentation/process/submitting-patches.rst | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst > index c091710..30dc00a 100644 > --- a/Documentation/process/submitting-patches.rst > +++ b/Documentation/process/submitting-patches.rst > @@ -510,7 +510,7 @@ tracking your trees, and to people trying to troubleshoot bugs in your > tree. > > > -12) When to use Acked-by:, Cc:, and Co-Developed-by: > +12) When to use Acked-by:, Cc:, and Co-developed-by: > ------------------------------------------------------- > > The Signed-off-by: tag indicates that the signer was involved in the > @@ -543,7 +543,7 @@ person it names - but it should indicate that this person was copied on the > patch. This tag documents that potentially interested parties > have been included in the discussion. > > -A Co-Developed-by: states that the patch was also created by another developer > +A Co-developed-by: states that the patch was also created by another developer > along with the original author. This is useful at times when multiple people > work on a single patch. Note, this person also needs to have a Signed-off-by: > line in the patch as well. > -- > 2.7.4 > -- Himanshu Jha Undergraduate Student Department of Electronics & Communication Guru Tegh Bahadur Institute of Technology ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] checkpatch: add Co-developed-by to signature tags 2018-12-14 17:35 [PATCH 0/2] Fix Co-Developed-by Jorge Ramirez-Ortiz 2018-12-14 17:35 ` [PATCH 1/2] docs: fix Co-Developed-by docs Jorge Ramirez-Ortiz @ 2018-12-14 17:35 ` Jorge Ramirez-Ortiz 2018-12-14 20:16 ` Andrew Morton 1 sibling, 1 reply; 7+ messages in thread From: Jorge Ramirez-Ortiz @ 2018-12-14 17:35 UTC (permalink / raw) To: jorge.ramirez-ortiz, joe, gregkh, himanshujha199640 Cc: apw, akpm, linux-kernel, niklas.cassel, corbet, linux-doc As per Documentation/process/submitting-patches, Co-developed-by is a valid signature. This commit removes the warning. Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> --- scripts/checkpatch.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 93e84c9..b737ca9 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -468,6 +468,7 @@ our $logFunctions = qr{(?x: our $signature_tags = qr{(?xi: Signed-off-by:| + Co-developed-by:| Acked-by:| Tested-by:| Reviewed-by:| -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] checkpatch: add Co-developed-by to signature tags 2018-12-14 17:35 ` [PATCH 2/2] checkpatch: add Co-developed-by to signature tags Jorge Ramirez-Ortiz @ 2018-12-14 20:16 ` Andrew Morton 2018-12-14 20:58 ` Himanshu Jha 2018-12-15 17:45 ` Joe Perches 0 siblings, 2 replies; 7+ messages in thread From: Andrew Morton @ 2018-12-14 20:16 UTC (permalink / raw) To: Jorge Ramirez-Ortiz Cc: joe, gregkh, himanshujha199640, apw, linux-kernel, niklas.cassel, corbet, linux-doc On Fri, 14 Dec 2018 18:35:28 +0100 Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> wrote: > As per Documentation/process/submitting-patches, Co-developed-by is a > valid signature. > I'm with Joe - I find this tag kinda useless and duplicative. But whatever. But as the documentation says, Co-developed-by must be accompanied by a Signed-off-by:. It would be helpful if checkpatch were to detect a failure to do this. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] checkpatch: add Co-developed-by to signature tags 2018-12-14 20:16 ` Andrew Morton @ 2018-12-14 20:58 ` Himanshu Jha 2018-12-15 17:45 ` Joe Perches 1 sibling, 0 replies; 7+ messages in thread From: Himanshu Jha @ 2018-12-14 20:58 UTC (permalink / raw) To: Andrew Morton Cc: Jorge Ramirez-Ortiz, joe, gregkh, apw, linux-kernel, niklas.cassel, corbet, linux-doc On Fri, Dec 14, 2018 at 12:16:34PM -0800, Andrew Morton wrote: > On Fri, 14 Dec 2018 18:35:28 +0100 Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> wrote: > > > As per Documentation/process/submitting-patches, Co-developed-by is a > > valid signature. > > > > I'm with Joe - I find this tag kinda useless and duplicative. But whatever. I'm fine either way, just the problem is: Co-developed-by Vs Co-Developed-by with checkpatch spitting out that it is not a valid signature. In the end, I can completely remove the tag from docs if it is useless and duplicative. -- Himanshu Jha Undergraduate Student Department of Electronics & Communication Guru Tegh Bahadur Institute of Technology ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] checkpatch: add Co-developed-by to signature tags 2018-12-14 20:16 ` Andrew Morton 2018-12-14 20:58 ` Himanshu Jha @ 2018-12-15 17:45 ` Joe Perches 1 sibling, 0 replies; 7+ messages in thread From: Joe Perches @ 2018-12-15 17:45 UTC (permalink / raw) To: Andrew Morton, Jorge Ramirez-Ortiz Cc: gregkh, himanshujha199640, apw, linux-kernel, niklas.cassel, corbet, linux-doc On Fri, 2018-12-14 at 12:16 -0800, Andrew Morton wrote: > On Fri, 14 Dec 2018 18:35:28 +0100 Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> wrote: > > > As per Documentation/process/submitting-patches, Co-developed-by is a > > valid signature. > > > > I'm with Joe - I find this tag kinda useless and duplicative. But whatever. > > But as the documentation says, Co-developed-by must be accompanied by a > Signed-off-by:. It would be helpful if checkpatch were to detect a > failure to do this. perhaps --- scripts/checkpatch.pl | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 93e84c9504a1..056d4b47ffaf 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2673,7 +2673,20 @@ sub process { WARN("BAD_SIGN_OFF", "Duplicate signature\n" . $herecurr); } else { - $signatures{$sig_nospace} = 1; + $signatures{$sig_nospace} = $herecurr; + } + } + +# Check for "Co-developed-by:" uses without equivalent "Signed-off-by:" +# $signatures keys are all lower case without spaces + if ($rawline =~ /^---$/) { + foreach my $sig (keys %signatures) { + next if ($sig !~ /^co-developed-by:(.*)/); + my $signer = $1; + if (!defined $signatures{"signed-off-by:$signer"}) { + WARN("CODEVELOPED_WITHOUT_SIGNOFF", + "Co-developed-by: used without Signed-off-by: by same person\n" . $signatures{$sig}); + } } } ^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-12-15 17:45 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-12-14 17:35 [PATCH 0/2] Fix Co-Developed-by Jorge Ramirez-Ortiz 2018-12-14 17:35 ` [PATCH 1/2] docs: fix Co-Developed-by docs Jorge Ramirez-Ortiz 2018-12-14 18:12 ` Himanshu Jha 2018-12-14 17:35 ` [PATCH 2/2] checkpatch: add Co-developed-by to signature tags Jorge Ramirez-Ortiz 2018-12-14 20:16 ` Andrew Morton 2018-12-14 20:58 ` Himanshu Jha 2018-12-15 17:45 ` Joe Perches
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).