[PATCH] linux-user, configure: improve syscall_nr.h dependencies checking

Laurent Vivier posted 1 patch 4 years, 1 month ago
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test checkpatch passed
Test FreeBSD passed
Test asan passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200323114116.163609-1-laurent@vivier.eu
configure | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
[PATCH] linux-user, configure: improve syscall_nr.h dependencies checking
Posted by Laurent Vivier 4 years, 1 month ago
This is mostly a fix for in-tree build.

It removes errors on .d directories:

  grep: ./.gitlab-ci.d: Is a directory
  grep: ./scripts/qemu-guest-agent/fsfreeze-hook.d: Is a directory

and improves performance by only checking <ARCH>-linux-user directories.

Reported-by: BALATON Zoltan <balaton@eik.bme.hu>
Tested-by: BALATON Zoltan <balaton@eik.bme.hu>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
 configure | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/configure b/configure
index 4f12cd01777e..da09c3589572 100755
--- a/configure
+++ b/configure
@@ -1910,9 +1910,9 @@ for arch in alpha hppa m68k xtensa sh4 microblaze arm ppc s390x sparc sparc64 \
     # remove the file if it has been generated in the source directory
     rm -f "${source_path}/linux-user/${arch}/syscall_nr.h"
     # remove the dependency files
-    find . -name "*.d" \
-           -exec grep -q "${source_path}/linux-user/${arch}/syscall_nr.h" {} \; \
-           -exec rm {} \;
+    test -d ${arch}-linux-user && find ${arch}-linux-user -type f -name "*.d" \
+         -exec grep -q "${source_path}/linux-user/${arch}/syscall_nr.h" {} \; \
+         -exec rm {} \;
 done
 
 if test -z "$python"
-- 
2.25.1


Re: [PATCH] linux-user, configure: improve syscall_nr.h dependencies checking
Posted by Philippe Mathieu-Daudé 4 years, 1 month ago
On 3/23/20 12:41 PM, Laurent Vivier wrote:
> This is mostly a fix for in-tree build.
> 
> It removes errors on .d directories:
> 
>    grep: ./.gitlab-ci.d: Is a directory
>    grep: ./scripts/qemu-guest-agent/fsfreeze-hook.d: Is a directory
> 
> and improves performance by only checking <ARCH>-linux-user directories.
> 
> Reported-by: BALATON Zoltan <balaton@eik.bme.hu>
> Tested-by: BALATON Zoltan <balaton@eik.bme.hu>
> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
> ---
>   configure | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/configure b/configure
> index 4f12cd01777e..da09c3589572 100755
> --- a/configure
> +++ b/configure
> @@ -1910,9 +1910,9 @@ for arch in alpha hppa m68k xtensa sh4 microblaze arm ppc s390x sparc sparc64 \
>       # remove the file if it has been generated in the source directory
>       rm -f "${source_path}/linux-user/${arch}/syscall_nr.h"
>       # remove the dependency files
> -    find . -name "*.d" \
> -           -exec grep -q "${source_path}/linux-user/${arch}/syscall_nr.h" {} \; \
> -           -exec rm {} \;
> +    test -d ${arch}-linux-user && find ${arch}-linux-user -type f -name "*.d" \
> +         -exec grep -q "${source_path}/linux-user/${arch}/syscall_nr.h" {} \; \
> +         -exec rm {} \;
>   done
>   
>   if test -z "$python"
> 

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>


Re: [PATCH] linux-user, configure: improve syscall_nr.h dependencies checking
Posted by Peter Maydell 4 years, 1 month ago
On Mon, 23 Mar 2020 at 11:42, Laurent Vivier <laurent@vivier.eu> wrote:
>
> This is mostly a fix for in-tree build.
>
> It removes errors on .d directories:
>
>   grep: ./.gitlab-ci.d: Is a directory
>   grep: ./scripts/qemu-guest-agent/fsfreeze-hook.d: Is a directory
>
> and improves performance by only checking <ARCH>-linux-user directories.
>
> Reported-by: BALATON Zoltan <balaton@eik.bme.hu>
> Tested-by: BALATON Zoltan <balaton@eik.bme.hu>
> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
> ---

Applied to master to fix the build errors for in-tree builds.

thanks
-- PMM