diff mbox series

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

Message ID 20200323114116.163609-1-laurent@vivier.eu
State New
Headers show
Series linux-user, configure: improve syscall_nr.h dependencies checking | expand

Commit Message

Laurent Vivier March 23, 2020, 11:41 a.m. UTC
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(-)

Comments

Philippe Mathieu-Daudé March 23, 2020, 11:45 a.m. UTC | #1
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>
Peter Maydell March 23, 2020, 3:38 p.m. UTC | #2
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
diff mbox series

Patch

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"