[PATCH] utils: checkstyle.py: Warn when no valid Signed-off-by line is found

[Milan Zamazal]

Laurent Pinchart <laurent.pinchart at ideasonboard.com> writes:

> All commits to libcamera must include a Signed-off-by line, and that
> rule is enforced through git hooks and CI. This however doesn't prevent
> patches from being submitted without an SoB tag, as noticed multiple
> times in the past. Extend the checkstyle.py trailer checker to issue a
> warning when the SoB line is missing to try and improve the situation.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Milan Zamazal <mzamazal at redhat.com>

Now how to check that checkstyle.py has been run before a patch is
submitted. ;-)

> ---
>  utils/checkstyle.py | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/utils/checkstyle.py b/utils/checkstyle.py
> index 4185c39ac811..7d480bdf4a2f 100755
> --- a/utils/checkstyle.py
> +++ b/utils/checkstyle.py
> @@ -493,6 +493,8 @@ class TrailersChecker(CommitChecker):
>      def check(cls, commit, top_level):
>          issues = []
>  
> +        sob_found = False
> +
>          for trailer in commit.trailers:
>              match = TrailersChecker.trailer_regex.fullmatch(trailer)
>              if not match:
> @@ -515,6 +517,12 @@ class TrailersChecker(CommitChecker):
>                  issues.append(CommitIssue(f"Malformed value '{value}' for commit trailer '{key}'"))
>                  continue
>  
> +            if key == 'Signed-off-by':
> +                sob_found = True
> +
> +        if not sob_found:
> +            issues.append(CommitIssue(f"No valid 'Signed-off-by' trailer found, see Documentation/contributing.rst"))
> +
>          return issues
>  
>  
>
> base-commit: 19bbca3c0b376ba0183f5db53472c8c46cd402b5