[v5] scripts/checkpatch: fix SPDX-License-Identifier detection

[PATCH v5 10/9] scripts/checkpatch.pl: mandate SPDX tag for Rust src files

Posted by Daniel P. Berrangé 6 months, 4 weeks ago

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 scripts/checkpatch.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 17b8db40ad..c0b36a85cd 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -1499,7 +1499,7 @@ sub process_end_of_file {
 	if ($fileinfo->{action} eq "new" &&
 	    !exists $fileinfo->{facts}->{sawspdx}) {
 		if ($fileinfo->{filenew} =~
-		    /(\.(c|h|py|pl|sh|json|inc)|Makefile.*)$/) {
+		    /(\.(c|h|py|pl|sh|json|inc|rs)|Makefile.*)$/) {
 			# source code files MUST have SPDX license declared
 			ERROR("New file '" . $fileinfo->{filenew} .
 			      "' requires 'SPDX-License-Identifier'");
-- 
2.49.0

Re: [PATCH v5 10/9] scripts/checkpatch.pl: mandate SPDX tag for Rust src files

Posted by Manos Pitsidianakis 6 months, 4 weeks ago

On Wed, May 21, 2025 at 12:06 PM Daniel P. Berrangé <berrange@redhat.com> wrote:
>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>  scripts/checkpatch.pl | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 17b8db40ad..c0b36a85cd 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -1499,7 +1499,7 @@ sub process_end_of_file {
>         if ($fileinfo->{action} eq "new" &&
>             !exists $fileinfo->{facts}->{sawspdx}) {
>                 if ($fileinfo->{filenew} =~
> -                   /(\.(c|h|py|pl|sh|json|inc)|Makefile.*)$/) {
> +                   /(\.(c|h|py|pl|sh|json|inc|rs)|Makefile.*)$/) {
>                         # source code files MUST have SPDX license declared
>                         ERROR("New file '" . $fileinfo->{filenew} .
>                               "' requires 'SPDX-License-Identifier'");
> --
> 2.49.0
>

Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>