<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 17, 2020 at 12:12 PM Aditya <<a href="mailto:yashsri421@gmail.com">yashsri421@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 16/12/20 9:59 pm, Aditya wrote:<br>
> On 16/12/20 6:27 pm, Lukas Bulwahn wrote:<br>
>> On Wed, Dec 16, 2020 at 1:54 PM Aditya <<a href="mailto:yashsri421@gmail.com" target="_blank">yashsri421@gmail.com</a>> wrote:<br>
>>><br>
>>> On 16/12/20 6:05 pm, Lukas Bulwahn wrote:<br>
>>>> On Wed, Dec 16, 2020 at 1:32 PM Aditya Srivastava <<a href="mailto:yashsri421@gmail.com" target="_blank">yashsri421@gmail.com</a>> wrote:<br>
>>>>><br>
>>>>> Currently checkpatch warns for long line in commit messages even for<br>
>>>>> URL lines.<br>
>>>>><br>
>>>>> An evaluation over v4.13..v5.8 showed that out of ~11000 warnings for<br>
>>>>> this class, around 790 are due to the line containing link.<br>
>>>>><br>
>>>>> E.g. running checkpatch on commit 3cde818cd02b ("ASoC: topology:<br>
>>>>> Consolidate how dtexts and dvalues are freed") reports this warning:<br>
>>>>><br>
>>>>> WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)<br>
>>>>> <a href="https://mailman.alsa-project.org/pipermail/alsa-devel/2019-January/144761.html" rel="noreferrer" target="_blank">https://mailman.alsa-project.org/pipermail/alsa-devel/2019-January/144761.html</a><br>
>>>>><br>
>>>>> Avoid giving users warning for character limit, instead suggest them to<br>
>>>>> prefix the URLs with "Link:"<br>
>>>>><br>
>>>><br>
>>>> Looks good to me. This might be also a good one for a fix option.<br>
>>>><br>
>>><br>
>>> I agree. Should I add the fix in this patch itself?<br>
>>><br>
>><br>
>> No, make two patches and send them as a patch series to the mailing list.<br>
>><br>
> Hi Lukas<br>
> So, I generated a list of lines, which are URLs and reported by<br>
> COMMIT_LOG_LONG_LINE here:<br>
> <a href="https://github.com/AdityaSrivast/kernel-tasks/blob/master/Task6/links/long_line_links.txt" rel="noreferrer" target="_blank">https://github.com/AdityaSrivast/kernel-tasks/blob/master/Task6/links/long_line_links.txt</a><br>
> <br>
> These occur in different forms:<br>
> <br>
> 1) A large part of these occur as \s*(http|https).<br>
> <br>
> 2) These also occur as [1]: https, [2]: https, etc<br>
> <br>
> 3) BugLink: https:// (e.g.: BugLink:<br>
> <a href="http://lkml.kernel.org/r/20180426045223.GA15307@dragonet.kaist.ac.kr" rel="noreferrer" target="_blank">http://lkml.kernel.org/r/20180426045223.GA15307@dragonet.kaist.ac.kr</a>)<br>
> <br>
> 4) References: https:// (e.g.: References:<br>
> <a href="http://mid.mail-archive.com/alpine.DEB.2.20.1905262211270.24390@whs-18.cs.helsinki.fi" rel="noreferrer" target="_blank">http://mid.mail-archive.com/alpine.DEB.2.20.1905262211270.24390@whs-18.cs.helsinki.fi</a>)<br>
> <br>
> 5) Link to v1: (E.g.: Link to v1:<br>
> <a href="https://patchwork.freedesktop.org/patch/msgid/20190228210939.83386-6-sean@poorly.run" rel="noreferrer" target="_blank">https://patchwork.freedesktop.org/patch/msgid/20190228210939.83386-6-sean@poorly.run</a>)<br>
> <br>
> 6) Datasheet (e.g.<br>
> Datasheet:<a href="http://www.winbond.com/resource-files/w25n01gv%20revl%20050918%20unsecured.pdf" rel="noreferrer" target="_blank">http://www.winbond.com/resource-files/w25n01gv%20revl%20050918%20unsecured.pdf</a>)<br>
> <br>
> 7) URL enclosed within brackets (E.g.:<br>
> (<a href="https://syzkaller.appspot.com/bug?id=ad7e0351fbc90535558514a71cd3edc11681997a)" rel="noreferrer" target="_blank">https://syzkaller.appspot.com/bug?id=ad7e0351fbc90535558514a71cd3edc11681997a)</a>.)<br>
> <br>
> 8) Link within sentences (E.g.: suite can be found at<br>
> <a href="https://github.com/Cyan4973/xxHash/blob/cf46e0c/xxhsum.c#L664" rel="noreferrer" target="_blank">https://github.com/Cyan4973/xxHash/blob/cf46e0c/xxhsum.c#L664</a>)<br>
> <br>
> etc.<br>
> <br>
> Among these I guess we can fix occurrences like 1st by prefixing with<br>
> "Link:". But I am not sure about the rest.<br>
> <br>
> What do you think?<br>
> <br>
<br>
I think we can only fix the lines with ^\s*(http|https|ftp) and should<br>
just give warning for the others. The user can then decide how they<br>
want to fix.<br>
<br>
Or should we give warning also for ^\s*(http|https|ftp) only?<br>
<br>
I guess we can send this patch to Joe and see what he thinks. This<br>
might give us a better idea for the fix.<br>
<br>
What do you think?<br>
<br></blockquote><div><br></div><div>Agree. If we summarize the use of hyperlinks in commit messages properly, we might also ask on the ksummit-discuss list for some feedback on "standardizing the use of hyperlinks in commit messages beyond Link-tags".</div><div><br></div><div>All those examples show that there are various different preferences but really no common agreed way that everyone uses.</div><div><br></div><div>Let us start the discussion with Joe and then see.</div><div><br></div><div>Lukas</div></div></div>