Since v6.3, checkpatch.pl now complains about the use of "Closes:" tags followed by a link [1]. It also complains if a "Reported-by:" tag is followed by a "Closes:" one [2]. As detailed in the first patch, this "Closes:" tag is used for a bit of time, mainly by DRM and MPTCP subsystems. It is used by some bug trackers to automate the closure of issues when a patch is accepted. Because this tag is used for a bit of time by different subsystems and it looks like it makes sense and it is useful for them, I didn't bother Linus to get his permission to continue using it. If you think this is necessary to do that up front, please tell me and I will be happy to ask for his agreement. The first patch updates the documentation to explain what is this "Closes:" tag and how/when to use it. The second patch modifies checkpatch.pl to stop complaining about it. The DRM maintainers and their mailing list have been added in Cc as they are probably interested by these two patches as well. [1] https://lore.kernel.org/all/3b036087d80b8c0e07a46a1dbaaf4ad0d018f8d5.1674217480.git.linux@xxxxxxxxxxxxx/ [2] https://lore.kernel.org/all/bb5dfd55ea2026303ab2296f4a6df3da7dd64006.1674217480.git.linux@xxxxxxxxxxxxx/ Signed-off-by: Matthieu Baerts <matthieu.baerts@xxxxxxxxxxxx> --- Matthieu Baerts (2): docs: process: allow Closes tags with links checkpatch: allow Closes tags with links Documentation/process/5.Posting.rst | 9 +++++++++ Documentation/process/submitting-patches.rst | 7 +++++++ scripts/checkpatch.pl | 16 ++++++++-------- 3 files changed, 24 insertions(+), 8 deletions(-) --- base-commit: 6015b1aca1a233379625385feb01dd014aca60b5 change-id: 20230314-doc-checkpatch-closes-tag-1731b57556b1 Best regards, -- Matthieu Baerts <matthieu.baerts@xxxxxxxxxxxx>
