This is an automated email from Gerrit.
"Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7572
-- gerrit
commit e773bc1932d4bdf9e4768a0e19bd5aa32b09ea12
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date: Sun Apr 2 20:47:41 2023 +0200
HACKING: add info on ignoring check-patch checks
Due to checkpatch internal state machine, the field 'Checkpatch-ignore:'
must be in the commit message before the 'Signed-off-by:' line.
Report it in the documentation and add that multiple 'Checkpatch-ignore:'
lines are allowed.
Change-Id: I770cdc4cb5b33bcf63c860c154ab3cbd4785ad20
Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>
diff --git a/HACKING b/HACKING
index be5699990e..d678c7b936 100644
--- a/HACKING
+++ b/HACKING
@@ -275,8 +275,12 @@ false errors.
Only for <em>exceptional cases</em>, it is allowed to submit patches
to Gerrit with the special field 'Checkpatch-ignore:' in the commit
message. This field will cause checkpatch to ignore the error types
-listed in the field, only for the patch itself.
+listed in the field, only for the patch itself, only for errors in
+the patch and not in its commit message.
The error type is printed by checkpatch on failure.
+The field 'Checkpatch-ignore:' must be added <em>before</em> the
+field 'Signed-off-by:', otherwise it is ignored.
+Multiple lines with field 'Checkpatch-ignore:' are allowed.
For example the names of Windows APIs mix lower and upper case chars,
in violation of OpenOCD coding style, triggering a 'CAMELCASE' error:
@code
--
