* [Linux-kernel-mentees] [PATCH] checkpatch: formatted email should have a whitespace separating address and comment
@ 2020-10-12 19:14 Dwaipayan Ray
2020-10-12 19:31 ` Dwaipayan Ray
0 siblings, 1 reply; 4+ messages in thread
From: Dwaipayan Ray @ 2020-10-12 19:14 UTC (permalink / raw)
To: lukas.bulwahn; +Cc: dwaipayanray1, linux-kernel-mentees
commit 0c01921 ("checkpatch: add new warnings to author signoff checks.")
introduced new checks for author sign off. The format_email procedure
was modified to add comment blocks to the formatted email. But no
space was added between email address and the mail comment.
This caused issues in some cases where the email was in the form
of "author@example.com (Comment block)". The formatted email removed
the space at the end of address.
An evaluation on checkpatch brought up this case. For example,
on commit 1129d31b55d5 ("ima: Fix ima digest hash table key calculation"),
the following warning was reported:
WARNING:BAD_SIGN_OFF: email address 'David.Laight@aculab.com (big endian
system concerns)' might be better as 'David.Laight@aculab.com(big endian
system concerns)'
Resolved this by adding a single space in between the address and comment
when the extracted comment is not empty.
Signed-off-by: Dwaipayan Ray <dwaipayanray1@gmail.com>
---
scripts/checkpatch.pl | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 124ff9432b51..6000addf1af3 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -1201,7 +1201,9 @@ sub format_email {
} else {
$formatted_email = "$name$name_comment <$address>";
}
- $formatted_email .= "$comment";
+ if ("$comment" ne "") {
+ $formatted_email .= " $comment";
+ }
return $formatted_email;
}
--
2.27.0
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [Linux-kernel-mentees] [PATCH] checkpatch: formatted email should have a whitespace separating address and comment
2020-10-12 19:14 [Linux-kernel-mentees] [PATCH] checkpatch: formatted email should have a whitespace separating address and comment Dwaipayan Ray
@ 2020-10-12 19:31 ` Dwaipayan Ray
2020-10-12 19:49 ` Lukas Bulwahn
0 siblings, 1 reply; 4+ messages in thread
From: Dwaipayan Ray @ 2020-10-12 19:31 UTC (permalink / raw)
To: Lukas Bulwahn; +Cc: linux-kernel-mentees
On Tue, Oct 13, 2020 at 12:45 AM Dwaipayan Ray <dwaipayanray1@gmail.com> wrote:
>
> commit 0c01921 ("checkpatch: add new warnings to author signoff checks.")
> introduced new checks for author sign off. The format_email procedure
> was modified to add comment blocks to the formatted email. But no
> space was added between email address and the mail comment.
>
> This caused issues in some cases where the email was in the form
> of "author@example.com (Comment block)". The formatted email removed
> the space at the end of address.
>
> An evaluation on checkpatch brought up this case. For example,
> on commit 1129d31b55d5 ("ima: Fix ima digest hash table key calculation"),
> the following warning was reported:
>
> WARNING:BAD_SIGN_OFF: email address 'David.Laight@aculab.com (big endian
> system concerns)' might be better as 'David.Laight@aculab.com(big endian
> system concerns)'
>
> Resolved this by adding a single space in between the address and comment
> when the extracted comment is not empty.
>
> Signed-off-by: Dwaipayan Ray <dwaipayanray1@gmail.com>
> ---
> scripts/checkpatch.pl | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 124ff9432b51..6000addf1af3 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -1201,7 +1201,9 @@ sub format_email {
> } else {
> $formatted_email = "$name$name_comment <$address>";
> }
> - $formatted_email .= "$comment";
> + if ("$comment" ne "") {
> + $formatted_email .= " $comment";
> + }
> return $formatted_email;
> }
>
> --
> 2.27.0
>
Found this issue while looking through the checkpatch records.
Investigated this after looking into Aditya's summary at
https://lore.kernel.org/linux-kernel-mentees/a1675970-9eb9-bd6a-31ff-d04f88ca30e2@gmail.com/
and definitely the space bug was affecting and causing some issues.
If it's alright, could I quickly send it over to Joe?
Thanks,
Dwaipayan.
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [Linux-kernel-mentees] [PATCH] checkpatch: formatted email should have a whitespace separating address and comment
2020-10-12 19:31 ` Dwaipayan Ray
@ 2020-10-12 19:49 ` Lukas Bulwahn
2020-10-12 20:05 ` Dwaipayan Ray
0 siblings, 1 reply; 4+ messages in thread
From: Lukas Bulwahn @ 2020-10-12 19:49 UTC (permalink / raw)
To: Dwaipayan Ray; +Cc: linux-kernel-mentees
[-- Attachment #1.1: Type: text/plain, Size: 2358 bytes --]
The commit message header should be in imperative and it should be shorter.
I think something up to sixty characters is preferred.
Lukas
On Mo., 12. Okt. 2020 at 21:31, Dwaipayan Ray <dwaipayanray1@gmail.com>
wrote:
> On Tue, Oct 13, 2020 at 12:45 AM Dwaipayan Ray <dwaipayanray1@gmail.com>
> wrote:
> >
> > commit 0c01921 ("checkpatch: add new warnings to author signoff checks.")
> > introduced new checks for author sign off. The format_email procedure
> > was modified to add comment blocks to the formatted email. But no
> > space was added between email address and the mail comment.
> >
> > This caused issues in some cases where the email was in the form
> > of "author@example.com (Comment block)". The formatted email removed
> > the space at the end of address.
> >
> > An evaluation on checkpatch brought up this case. For example,
> > on commit 1129d31b55d5 ("ima: Fix ima digest hash table key
> calculation"),
> > the following warning was reported:
> >
> > WARNING:BAD_SIGN_OFF: email address 'David.Laight@aculab.com (big endian
> > system concerns)' might be better as 'David.Laight@aculab.com(big endian
> > system concerns)'
> >
> > Resolved this by adding a single space in between the address and comment
> > when the extracted comment is not empty.
> >
> > Signed-off-by: Dwaipayan Ray <dwaipayanray1@gmail.com>
> > ---
> > scripts/checkpatch.pl | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> > index 124ff9432b51..6000addf1af3 100755
> > --- a/scripts/checkpatch.pl
> > +++ b/scripts/checkpatch.pl
> > @@ -1201,7 +1201,9 @@ sub format_email {
> > } else {
> > $formatted_email = "$name$name_comment <$address>";
> > }
> > - $formatted_email .= "$comment";
> > + if ("$comment" ne "") {
> > + $formatted_email .= " $comment";
> > + }
> > return $formatted_email;
> > }
> >
> > --
> > 2.27.0
> >
>
> Found this issue while looking through the checkpatch records.
>
> Investigated this after looking into Aditya's summary at
>
> https://lore.kernel.org/linux-kernel-mentees/a1675970-9eb9-bd6a-31ff-d04f88ca30e2@gmail.com/
> and definitely the space bug was affecting and causing some issues.
>
> If it's alright, could I quickly send it over to Joe?
>
> Thanks,
> Dwaipayan.
>
[-- Attachment #1.2: Type: text/html, Size: 3911 bytes --]
[-- Attachment #2: Type: text/plain, Size: 201 bytes --]
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-10-12 20:05 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-12 19:14 [Linux-kernel-mentees] [PATCH] checkpatch: formatted email should have a whitespace separating address and comment Dwaipayan Ray
2020-10-12 19:31 ` Dwaipayan Ray
2020-10-12 19:49 ` Lukas Bulwahn
2020-10-12 20:05 ` Dwaipayan Ray
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).