[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 01/19] scripts/clean-includes: Fully skip / ignore files
From: |
Eric Blake |
Subject: |
Re: [PATCH v4 01/19] scripts/clean-includes: Fully skip / ignore files |
Date: |
Fri, 27 Jan 2023 15:29:33 -0600 |
User-agent: |
NeoMutt/20220429 |
On Thu, Jan 19, 2023 at 07:59:41AM +0100, Markus Armbruster wrote:
...
>
> Fix the script to fully skip files.
>
> Fixes: fd3e39a40ca2ee26b09a5de3149af8b056b85233
> Fixes: d66253e46ae2b9c36a9dd90b2b74c0dfa5804b22
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
> scripts/clean-includes | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/scripts/clean-includes b/scripts/clean-includes
> index d37bd4f692..86944f27fc 100755
> --- a/scripts/clean-includes
> +++ b/scripts/clean-includes
> @@ -111,6 +111,7 @@ cat >"$COCCIFILE" <<EOT
> )
> EOT
>
> +files=
> for f in "$@"; do
> case "$f" in
> *.c.inc)
> @@ -144,6 +145,7 @@ for f in "$@"; do
> continue
> ;;
> esac
> + files="$files $f"
Bash's += might perform faster here, but this is a #!/bin/sh script.
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
[PATCH v4 14/19] block: Clean up includes, Markus Armbruster, 2023/01/19