<div dir="ltr">On Sun, Jul 26, 2020 at 4:43 PM Lukas Bulwahn <<a href="mailto:lukas.bulwahn@gmail.com">lukas.bulwahn@gmail.com</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Commit message is wrong.<br>
<br>
On Sun, 26 Jul 2020, Nachiket Naganure wrote:<br>
<br>
> Checkpatch.pl outputs warnings for commit description lines that exceed<br>
> beyond 75 characters. Sometimes these lines might contain signature tags<br>
> that cannot be shortened. The patch checks if the line has any valid<br>
> signature tags and disables the warning in such a case.<br>
> <br>
> For instance in commit ID ac854131d984, the patch<br>
> contains a Reported by tag line which exceeds 75 chars. And there is no<br>
> valid way to shorten the length.<br>
> <br>
> To fix this, we need to check if the $line variable has any valid<br>
> signature tags which are already stored in $signature_tags variable.<br>
> <br>
<br>
Read the documentation on commit messages. Think about the things you are <br>
doing wrong in your commit message.<br>
<br>
> Signed-off-by: Nachiket Naganure <<a href="mailto:nachiketun8@gmail.com" target="_blank">nachiketun8@gmail.com</a>><br>
> ---<br>
>  scripts/<a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a> | 2 ++<br>
>  1 file changed, 2 insertions(+)<br>
> <br>
> diff --git a/scripts/<a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a> b/scripts/<a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a><br>
> index 197436b20288..b6d5e2a8fce8 100755<br>
> --- a/scripts/<a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a><br>
> +++ b/scripts/<a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a><br>
> @@ -2806,6 +2806,8 @@ sub process {<br>
>                                       # filename then :<br>
>                     $line =~ /^\s*(?:Fixes:|Link:)/i ||<br>
>                                       # A Fixes: or Link: line<br>
> +          $line =~ /$signature_tags/ ||<br>
> +          # Checks for signature_tags<br>
<br>
Change looks more reasonable now.<br>
Indentation is broken.<br>
<br>
<br>
Maybe two or three iterations on this patch and it is getting ready for <br>
submission.<br>
<br>
Lukas<br>
<br>
>                     $commit_log_possible_stack_dump)) {<br>
>                       WARN("COMMIT_LOG_LONG_LINE",<br>
>                            "Possible unwrapped commit description (prefer a maximum 75 chars per line)\n" . $herecurr);<br>
> -- <br>
> 2.25.1<br>
> <br>
><br></blockquote><div>Checkpatch: Disable length check warning for signature tag<br><br>Disable commit log length check in case of the signature tag. If the commit<br>log line has valid signature tags such as "Reported-and-tested-by" with<br>more than 75 characters, it suppresses the long line length warning.<br><br>For instance in Commit ac854131d984 (USB: core: Fix misleading driver<br></div><div>bug report), the corresponding contains a "Reported by" tag line which<br>exceeds 75 chars. And there is no valid way to shorten the length.</div><div><br></div><div><br></div><div>The modified commit message is written above. Changed the language to<br></div><div>imperative and shortened the subject line. Fixed the commit ID reference.</div><div>Are there any other changes required?<br></div></div></div>