On 25 Mar 2025, at 16:11, Aaron Conole wrote:
> Checkpatch has the ability to scan source files rather than exclusively
> scanning patch files. In this mode, some checks get disabled (ex: all
> of the header and commit message parsing) and the display modes are
> adjusted to simply print the file and line, rather than including the patch
> line details.
>
> When checkpatch was updated for subject parsing, it was inadvertently
> ignoring the source file mode. This means any checks run against a source
> file will always print a bogus subject suggestion, and warning. Fix this
> by not printing these warnings when in source file mode. This should have
> no effect on checkpatch against actual patch files.
>
> Fixes: 1b8fa4a66aa4 ("checkpatch: Add checks for the subject line.")
> Signed-off-by: Aaron Conole <[email protected]>
Did some testing, and the change looks good to me.
Acked-by: Eelco Chaudron <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev