From patchwork Mon Apr 3 16:23:45 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Matthieu Baerts X-Patchwork-Id: 7648 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp2428266vqo; Mon, 3 Apr 2023 09:26:45 -0700 (PDT) X-Google-Smtp-Source: AKy350Y8STmUJ7zqfD6FmU8viJJeHvuWeh3y5zd6zU8kPgWFJmcXDi1Ve8nw9w136g1plzxgGN4y X-Received: by 2002:a05:6a20:baa4:b0:d9:7af9:6a82 with SMTP id fb36-20020a056a20baa400b000d97af96a82mr19808586pzb.9.1680539205141; Mon, 03 Apr 2023 09:26:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1680539205; cv=none; d=google.com; s=arc-20160816; b=Ery35bUZFehwcfUI56vkmHYAS5e79HS2QzPLCDifkZEBg8h8vIGlKcqjKVzgrmNheJ 9cykZa/KFQUegd+b6SArBc4hgw8xZQ/xOIlJ/pRGzD7YpRsOzYg3tGGCq7SNyP1twurc ykV8/MS4yQa29PV3o21RkbrdzzTiwCID2gqQw2fKb3EoroSFSI0AZaPPinTVBW+Fl5Gn HVh6eC+qOCAg5B87EXPKOO6EtZhNU28/MUsT3ujLChM7RKsIFgFiR/O/emRW+SGe8h+M CmBMaepVnMF8imdZ333vemkIRNNlu4kxeNQp0ci27W+eqdTIY3WCMnmJixlUUMlAkUW4 mxag== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:content-transfer-encoding:mime-version :message-id:date:subject:from:dkim-signature; bh=E841ISFmhdikW8veVmziJDv7bwLBR4jkaGT4dRFLTmM=; b=W+XLpPx4UjocgsGwkARmTzgsj1C9hlaawKhRI+oR9KsPClvQiPpYbmljBXxDtCRarj 8W4SJ6FojeIn+U0/JpZmkPYEBRJt0DQ8MESMLyAT850tRkEYPyBtBQFSUxxGnQrI7GmX hdqOc8rPy9BMVqN7G0+LzwHApDZ4Qgm4M5TTtqakHEPyrrmNs2nBCm/MD444eKzsv38i LhOyY+HBH7lJ9oySwzXn/Po+/EjNri/jbHyFyydzVrs1dAUL+aFZYqQJe4XPDQhOglks XrD4V8FJSJRRLJGUU3KRXQW/AJI9os57L/24oWYZ/JMRg02873bjlS2OOVYXToM7UjWw DTLw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@tessares.net header.s=google header.b=wqswj6Sj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=tessares.net Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id a193-20020a621aca000000b005a9cfb6ee90si8509415pfa.84.2023.04.03.09.26.32; Mon, 03 Apr 2023 09:26:45 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@tessares.net header.s=google header.b=wqswj6Sj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=tessares.net Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232720AbjDCQYC (ORCPT + 99 others); Mon, 3 Apr 2023 12:24:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53902 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232555AbjDCQXw (ORCPT ); Mon, 3 Apr 2023 12:23:52 -0400 Received: from mail-wm1-x32a.google.com (mail-wm1-x32a.google.com [IPv6:2a00:1450:4864:20::32a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F3C76E9 for ; Mon, 3 Apr 2023 09:23:50 -0700 (PDT) Received: by mail-wm1-x32a.google.com with SMTP id j1-20020a05600c1c0100b003f04da00d07so1779082wms.1 for ; Mon, 03 Apr 2023 09:23:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tessares.net; s=google; t=1680539029; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=E841ISFmhdikW8veVmziJDv7bwLBR4jkaGT4dRFLTmM=; b=wqswj6Sj7OxB5slUYnbQiBGf7pcDxzHiUAiu4HRxJCGMrepldM0RavmChoH7Yz890K 4fid8IFvZbdlfJPjI6sHWSjXFkQb58n32OM7Sh6hnhdhfjtuwlfMFIW9Ou4gnvQ4VB+j 8IqM2SJCnv91z6hjNlhmXa04eRLjQgcpgMl+qBn5K2whrIk12/AQ2XLRFcPquH7wp0da sNxcKUg035BTn+K1At55esFQFrJCOf9aLS2b/MwSnvtBGcrd16WTPV8RhoO+Anr78hw3 k0SpdwRMk2l2XWh+NFtUnndM2uZQfAOQP3Ji8k+t2OVUIoEWmXPFr8aWKWu70U3jvPgi jjGg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1680539029; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=E841ISFmhdikW8veVmziJDv7bwLBR4jkaGT4dRFLTmM=; b=67+aMQQacmw4gJvETC2gCB6t9tepUrZc5g2IBgxr/5fmTopfUP3UWjwA+MqQIkJlST FA7N/N5mK9pHiW0BdbHqXRDTdLtcEb/mytaPg4Ef0xl6Xx+719IpQBG4RIvohcKAKsws u+PfjkvWhtkAVYCsPKoIeRlMuBTvAy0eN0xuYrfNMbrLaTwNVlmenf3G9HTOctjFaUgM Y6yNypP/HOFDgpBGJmAsJ8HYE3f5qgCuKdW2Vw9sOM6VB6aCkedUS0cuJH0Z/qYMKEgT N5F4kv5MGoI4znVbIkGjtaVOEN0F7gYm71Rft4EfrlvRod63KtnlIoux6ltEdFAkZVU2 7EFw== X-Gm-Message-State: AAQBX9fCrW8nD+OQ+WDE97r1XdX15Jlk12rcIGeWQUcbhOfotpeUEnT/ Ax9pNy6g4z9AJC4wVMzybdi3cA== X-Received: by 2002:a1c:7c17:0:b0:3eb:39e0:3530 with SMTP id x23-20020a1c7c17000000b003eb39e03530mr8611wmc.41.1680539029403; Mon, 03 Apr 2023 09:23:49 -0700 (PDT) Received: from vdi08.nix.tessares.net (static.219.156.76.144.clients.your-server.de. [144.76.156.219]) by smtp.gmail.com with ESMTPSA id s7-20020a05600c45c700b003ee2a0d49dbsm19829116wmo.25.2023.04.03.09.23.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 03 Apr 2023 09:23:49 -0700 (PDT) From: Matthieu Baerts Subject: [PATCH v4 0/5] docs & checkpatch: allow Closes tags with links Date: Mon, 03 Apr 2023 18:23:45 +0200 Message-Id: <20230314-doc-checkpatch-closes-tag-v4-0-d26d1fa66f9f@tessares.net> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAJH9KmQC/43OwU7DMAwG4FeZciZQJ22zceI9EAfHcZeILZ3iU IGmvjvpjohDj78tf7/vSrgkFvV6uKvCS5I05xb6p4OiiPnMOoWWlemM7Sz0OsykKTJ93rBS1HS ZhUVXPGtwFvzghmH0oNq9R2HtC2aKm3BJ+UteriiVy7a+FZ7S96P6/aPlmKTO5efxyQLbdE/pA rrT4I+2c4ZPeMK3yiJYWJ4zV7XBi9mNmYZNPfbgjiNMY/gHs7sx27AAPtBkgRzQH2xd11+8MRZ yfgEAAA== To: Jonathan Corbet , Andy Whitcroft , Joe Perches , Dwaipayan Ray , Lukas Bulwahn , =?utf-8?q?Kai_Wasserb=C3=A4ch?= , Thorsten Leemhuis , Andrew Morton , David Airlie , Daniel Vetter , Konstantin Ryabitsev , Bagas Sanjaya , Linus Torvalds Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, mptcp@lists.linux.dev, Matthieu Baerts X-Mailer: b4 0.12.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=3735; i=matthieu.baerts@tessares.net; h=from:subject:message-id; bh=0RP36hifT0KwybB2xAVif6HIHF0lZc8UPp6cjui9F1U=; b=owEBbQKS/ZANAwAIAfa3gk9CaaBzAcsmYgBkKv2T3N2a4Z8KjSF9P80Pqivkyx6Uum+l6DjOy q8dSYXIw3GJAjMEAAEIAB0WIQToy4X3aHcFem4n93r2t4JPQmmgcwUCZCr9kwAKCRD2t4JPQmmg cx55EACT3p2scRebAISag6Axflm7qLP3qvV6LJ8j00BebIZ2VJxMPl+B8ytiPj0LytRWucJ6/zc ifm1ejrc/zQw8ihSzBYCs2fyE5QNNxlu2ZmLiqKqg+/b1w9oAjRm/GnZJtrr0LbFoag9IN3FioA Wu9wb9KHEHZjiOZhjazkkUYWAaCYk1SnHa2c0ptgUU+3Hj/uQeFL5D/oUe8KcW/rsaxID8/7LIK pZ7tB29Jn3rP/MP5eg+Rn5fZuWXKLfSZKb0e38/WCr4JkGTGBvxwnd/QAlugn4pL/pYGd2xQBKS l3Sqpm99OUjmuigxyj4UQggVAA3IDK/38K5t+u45eaNeLui9Ug7Fn9nIyjOfTshbRMYGIlmhfyg LMD+9meoQcoNrM/09elq9vnTn1twB2jDp8MVaM0KMMU/LIfV7D9GlKzaCyYbYXA7raCBAVqt+MX H65pBQ30bzlwJj5tTzaQZ6poRLif3AMNWMJw7P7ZIYEtZWyl2zFe7DJ8vGyx9Q//gZiHviaFKvZ TbedcTLa6dttig7P0QRUCPoJGdudZUekuj1keJ2J70Q8vgtVlwnkTSBxkstbhClKrtBHM0XsLMN HGLg9AFa6xS8XW+/inYu2uuR0x+CFrPxXLo9gHBqGoW1WFFCWJ2bR3i4n5QQhuoY+gM/OwEQTu3 WQ8OTHBQd+nHEcQ== X-Developer-Key: i=matthieu.baerts@tessares.net; a=openpgp; fpr=E8CB85F76877057A6E27F77AF6B7824F4269A073 X-Spam-Status: No, score=-0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1762173077283145626?= X-GMAIL-MSGID: =?utf-8?q?1762173077283145626?= 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. It is even planned to use this tag with bugzilla.kernel.org [3]. 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@leemhuis.info/ [2] https://lore.kernel.org/all/bb5dfd55ea2026303ab2296f4a6df3da7dd64006.1674217480.git.linux@leemhuis.info/ [3] https://lore.kernel.org/linux-doc/20230315181205.f3av7h6owqzzw64p@meerkat.local/ Signed-off-by: Matthieu Baerts Acked-by: Joe Perches --- Note: After having re-read the comments from the v1, it is still unclear to me if this "Closes:" can be accepted or not. But because it seems that the future Bugzilla bot for kernel.org and regzbot would like to use it as well, I'm sending here new versions. I'm sorry if I misunderstood the comments from v1. Please tell me if I did. Changes in v4: - Patches 1/5, 3/5 and 4/5 have been added to ask using the "Closes" tag instead of the "Link" one for any bug reports. (Thorsten) - The Fixes tags have been removed from patch 4/5. (Joe) - The "Reported-by being followed by a link tag" check is now only looking for the tag, not the URL which is done elsewhere in patch 5/5. (Thorsten) - A new patch has been added to fix a small issues in checkpatch.pl when checking if "Reported-by:" tag is on the last line. - Link to v3: https://lore.kernel.org/r/20230314-doc-checkpatch-closes-tag-v3-0-d1bdcf31c71c@tessares.net Changes in v3: - Patch 1/4 now allow using the "Closes" tag with any kind of bug reports, as long as the link is public. (Thorsten) - The former patch 2/2 has been split in two: first to use a list for the different "link" tags (Joe). Then to allow the 'Closes' tag. - A new patch has been added to let checkpatch.pl checking if "Closes" and "Links" are used with a URL. - Link to v2: https://lore.kernel.org/r/20230314-doc-checkpatch-closes-tag-v2-0-f4a417861f6d@tessares.net Changes in v2: - The text on patch 1/2 has been reworked thanks to Jon, Bagas and Thorsten. See the individual changelog on the patch for more details. - Private bug trackers and invalid URLs are clearly marked as forbidden to avoid being misused. (Linus) - Rebased on top of Linus' repo. - Link to v1: https://lore.kernel.org/r/20230314-doc-checkpatch-closes-tag-v1-0-1b83072e9a9a@tessares.net --- Matthieu Baerts (5): docs: process: allow Closes tags with links checkpatch: don't print the next line if not defined checkpatch: use a list of "link" tags checkpatch: allow Closes tags with links checkpatch: check for misuse of the link tags Documentation/process/5.Posting.rst | 22 ++++++++++---- Documentation/process/submitting-patches.rst | 26 +++++++++++------ scripts/checkpatch.pl | 43 ++++++++++++++++++++++------ 3 files changed, 70 insertions(+), 21 deletions(-) --- base-commit: 7e364e56293bb98cae1b55fd835f5991c4e96e7d change-id: 20230314-doc-checkpatch-closes-tag-1731b57556b1 Best regards,