On Thu, 15 May 2025 at 14:59, Daniel P. Berrangé <[email protected]> wrote:
>
> The ACPI test data check needs to analyse a list of all files in a
> commit, so can use the new hook for processing the file list.
>
> Signed-off-by: Daniel P. Berrangé <[email protected]>
> ---
> scripts/checkpatch.pl | 61 ++++++++++++++++++++-----------------------
> 1 file changed, 29 insertions(+), 32 deletions(-)
> @@ -1770,9 +1769,7 @@ sub process {
> $line =~ /^rename (?:from|to) [\w\/\.\-]+\s*$/ ||
> ($line =~
> /\{\s*([\w\/\.\-]*)\s*\=\>\s*([\w\/\.\-]*)\s*\}/ &&
> (defined($1) || defined($2)))) &&
> - !(($realfile ne '') &&
> - defined($acpi_testexpected) &&
> - ($realfile eq $acpi_testexpected))) {
> + $realfile !~ m#^tests/data/acpi/#) {
Is the indentation off on this line?
> $reported_maintainer_file = 1;
> WARN("added, moved or deleted file(s), does
> MAINTAINERS need updating?\n" . $herecurr);
> }
> --
Otherwise
Reviewed-by: Peter Maydell <[email protected]>
thanks
-- PMM