diff options
| author | Nabih Estefan <nabihestefan@google.com> | 2025-09-22 10:36:48 +0100 |
|---|---|---|
| committer | Alex Bennée <alex.bennee@linaro.org> | 2025-09-26 09:55:19 +0100 |
| commit | 24e51fc773b63bd222e61226431f3b32ac9915c2 (patch) | |
| tree | 009bc43a9a8389052bc4b1d0ed132efd4db5eda5 /scripts | |
| parent | 5a449678d6594b70d3ead9ad78b47d76eb95a91e (diff) | |
| download | focaccia-qemu-24e51fc773b63bd222e61226431f3b32ac9915c2.tar.gz focaccia-qemu-24e51fc773b63bd222e61226431f3b32ac9915c2.zip | |
checkpatch: Ignore removed lines in license check
When running the license check, if we are updating a license it is possible for the checkpatch script to test against old license lines instead of newer ones, since the removal lines appear before the addition lines in a .patch file. Fix this by skipping over lines that start with "-" in the checkpatch script. Signed-off-by: Nabih Estefan <nabihestefan@google.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Message-ID: <20250916165928.10048-1-nabihestefan@google.com> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-ID: <20250922093711.2768983-4-alex.bennee@linaro.org>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/checkpatch.pl | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 91616c974f..40b6955c69 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -1816,7 +1816,8 @@ sub process { } # Check SPDX-License-Identifier references a permitted license - if ($rawline =~ m,SPDX-License-Identifier: (.*?)(\*/)?\s*$,) { + if (($rawline =~ m,SPDX-License-Identifier: (.*?)(\*/)?\s*$,) && + $rawline !~ /^-/) { $fileinfo->{facts}->{sawspdx} = 1; &checkspdx($realfile, $1); } |