If a patch has been reported by multiple individuals, allow a single
Closes tag.
Given this patch:
: Reported-by: kernel test robot <lkp@intel.com>
: Closes: https://lore.kernel.org/r/202411221147.N6w23gDo-lkp@intel.com/
: Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
: Closes: https://lore.kernel.org/r/202411221147.N6w23gDo-lkp@intel.com/
Allow also this:
: Reported-by: kernel test robot <lkp@intel.com>
: Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
: Closes: https://lore.kernel.org/r/202411221147.N6w23gDo-lkp@intel.com/
Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
---
scripts/checkpatch.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 9eed3683ad76..d6a79993fdf1 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -3189,7 +3189,7 @@ sub process {
if (!defined $lines[$linenr]) {
WARN("BAD_REPORTED_BY_LINK",
"Reported-by: should be immediately followed by Closes: with a URL to the report\n" . $herecurr . "\n");
- } elsif ($rawlines[$linenr] !~ /^closes:\s*/i) {
+ } elsif ($rawlines[$linenr] !~ /^closes:\s*/i && $rawlines[$linenr] !~ /^reported(?:|-and-tested)-by:\s*/i) {
WARN("BAD_REPORTED_BY_LINK",
"Reported-by: should be immediately followed by Closes: with a URL to the report\n" . $herecurr . $rawlines[$linenr] . "\n");
}
---
base-commit: 40384c840ea1944d7c5a392e8975ed088ecf0b37
change-id: 20241202-multi-close-3e23042d5185
Best regards,
--
Ricardo Ribalda <ribalda@chromium.org>