diff mbox series

[PULL,26/54] configure: unify creation of cross-compilation Makefiles

Message ID 20221004130138.2299307-27-alex.bennee@linaro.org
State Accepted
Commit 0825cae023133871dfbd803cae9f004cb7447a48
Headers show
Series [PULL,01/54] scripts/ci/setup: ninja missing from build-environment | expand

Commit Message

Alex Bennée Oct. 4, 2022, 1:01 p.m. UTC
From: Paolo Bonzini <pbonzini@redhat.com>

Let write_target_makefile handle both host and container cross compilers.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220929114231.583801-27-alex.bennee@linaro.org>

Comments

Stefan Hajnoczi Oct. 6, 2022, 8:33 p.m. UTC | #1
On Tue, Oct 04, 2022 at 02:01:10PM +0100, Alex Bennée wrote:
> From: Paolo Bonzini <pbonzini@redhat.com>
> 
> Let write_target_makefile handle both host and container cross compilers.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> Message-Id: <20220929114231.583801-27-alex.bennee@linaro.org>

I think this commit breaks bisection:

  configure: line 2498: syntax error near unexpected token `('
  configure: line 2498: `      echo "run-tcg-tests-$target: $qemu\$(EXESUF)" >> Makefile.prereqs'

It has already been merged into qemu.git/master, so there's no fixing it
now. I wanted to leave a comment in case someone hits this problem in
the future and wonders what's going on.

I had to use git bisect skip.

Stefan

> 
> diff --git a/configure b/configure
> index cbeac99b2b..8b495d4453 100755
> --- a/configure
> +++ b/configure
> @@ -2157,51 +2157,49 @@ probe_target_compiler() {
>  
>  write_target_makefile() {
>    echo "EXTRA_CFLAGS=$target_cflags"
> -  if test -n "$target_cc"; then
> -    echo "CC=$target_cc"
> -    echo "CCAS=$target_ccas"
> -  fi
> -  if test -n "$target_ar"; then
> -    echo "AR=$target_ar"
> -  fi
> -  if test -n "$target_as"; then
> -    echo "AS=$target_as"
> -  fi
> -  if test -n "$target_ld"; then
> -    echo "LD=$target_ld"
> -  fi
> -  if test -n "$target_nm"; then
> -    echo "NM=$target_nm"
> -  fi
> -  if test -n "$target_objcopy"; then
> -    echo "OBJCOPY=$target_objcopy"
> -  fi
> -  if test -n "$target_ranlib"; then
> -    echo "RANLIB=$target_ranlib"
> -  fi
> -  if test -n "$target_strip"; then
> -    echo "STRIP=$target_strip"
> -  fi
> -}
> -
> -write_container_target_makefile() {
> -  echo "$1: docker-image-$container_image" >> Makefile.prereqs
> -  echo "EXTRA_CFLAGS=$target_cflags"
> -  if test -n "$container_cross_cc"; then
> -    echo "CC=$docker_py cc --cc $container_cross_cc -i qemu/$container_image -s $source_path --"
> -    echo "CCAS=$docker_py cc --cc $container_cross_cc -i qemu/$container_image -s $source_path --"
> +  if test -z "$target_cc" && test -z "$target_as"; then
> +    test -z "$container_image" && error_exit "Internal error: could not find cross compiler for $1?"
> +    echo "$1: docker-image-$container_image" >> Makefile.prereqs
> +    if test -n "$container_cross_cc"; then
> +      echo "CC=$docker_py cc --cc $container_cross_cc -i qemu/$container_image -s $source_path --"
> +      echo "CCAS=$docker_py cc --cc $container_cross_cc -i qemu/$container_image -s $source_path --"
> +    fi
> +    echo "AR=$docker_py cc --cc $container_cross_ar -i qemu/$container_image -s $source_path --"
> +    echo "AS=$docker_py cc --cc $container_cross_as -i qemu/$container_image -s $source_path --"
> +    echo "LD=$docker_py cc --cc $container_cross_ld -i qemu/$container_image -s $source_path --"
> +    echo "NM=$docker_py cc --cc $container_cross_nm -i qemu/$container_image -s $source_path --"
> +    echo "OBJCOPY=$docker_py cc --cc $container_cross_objcopy -i qemu/$container_image -s $source_path --"
> +    echo "RANLIB=$docker_py cc --cc $container_cross_ranlib -i qemu/$container_image -s $source_path --"
> +    echo "STRIP=$docker_py cc --cc $container_cross_strip -i qemu/$container_image -s $source_path --"
> +  else
> +    if test -n "$target_cc"; then
> +      echo "CC=$target_cc"
> +      echo "CCAS=$target_ccas"
> +    fi
> +    if test -n "$target_ar"; then
> +      echo "AR=$target_ar"
> +    fi
> +    if test -n "$target_as"; then
> +      echo "AS=$target_as"
> +    fi
> +    if test -n "$target_ld"; then
> +      echo "LD=$target_ld"
> +    fi
> +    if test -n "$target_nm"; then
> +      echo "NM=$target_nm"
> +    fi
> +    if test -n "$target_objcopy"; then
> +      echo "OBJCOPY=$target_objcopy"
> +    fi
> +    if test -n "$target_ranlib"; then
> +      echo "RANLIB=$target_ranlib"
> +    fi
> +    if test -n "$target_strip"; then
> +      echo "STRIP=$target_strip"
> +    fi
>    fi
> -  echo "AR=$docker_py cc --cc $container_cross_ar -i qemu/$container_image -s $source_path --"
> -  echo "AS=$docker_py cc --cc $container_cross_as -i qemu/$container_image -s $source_path --"
> -  echo "LD=$docker_py cc --cc $container_cross_ld -i qemu/$container_image -s $source_path --"
> -  echo "NM=$docker_py cc --cc $container_cross_nm -i qemu/$container_image -s $source_path --"
> -  echo "OBJCOPY=$docker_py cc --cc $container_cross_objcopy -i qemu/$container_image -s $source_path --"
> -  echo "RANLIB=$docker_py cc --cc $container_cross_ranlib -i qemu/$container_image -s $source_path --"
> -  echo "STRIP=$docker_py cc --cc $container_cross_strip -i qemu/$container_image -s $source_path --"
>  }
>  
> -
> -
>  ##########################################
>  # check for vfio_user_server
>  
> @@ -2560,15 +2558,9 @@ for target in $target_list; do
>        ;;
>    esac
>  
> -  probe_target_compiler $target
> -  if test $got_cross_cc = yes; then
> -      write_target_makefile >> "$config_target_mak"
> -  elif test -n "$container_image"; then
> -      build_static=y
> -      write_container_target_makefile build-tcg-tests-$target >> "$config_target_mak"
> -      got_cross_cc=yes
> -  fi
> -  if test $got_cross_cc = yes; then
> +  if probe_target_compiler $target || test -n "$container_image"; then
> +      test -n "$container_image" && build_static=y
> +      write_target_makefile "build-tcg-tests-$target >> "$config_target_mak"
>        mkdir -p "tests/tcg/$target"
>        ln -sf "$source_path/tests/tcg/Makefile.target" "tests/tcg/$target/Makefile"
>        ln -sf "../config-$target.mak" "tests/tcg/$target/config-target.mak"
> -- 
> 2.34.1
>
diff mbox series

Patch

diff --git a/configure b/configure
index cbeac99b2b..8b495d4453 100755
--- a/configure
+++ b/configure
@@ -2157,51 +2157,49 @@  probe_target_compiler() {
 
 write_target_makefile() {
   echo "EXTRA_CFLAGS=$target_cflags"
-  if test -n "$target_cc"; then
-    echo "CC=$target_cc"
-    echo "CCAS=$target_ccas"
-  fi
-  if test -n "$target_ar"; then
-    echo "AR=$target_ar"
-  fi
-  if test -n "$target_as"; then
-    echo "AS=$target_as"
-  fi
-  if test -n "$target_ld"; then
-    echo "LD=$target_ld"
-  fi
-  if test -n "$target_nm"; then
-    echo "NM=$target_nm"
-  fi
-  if test -n "$target_objcopy"; then
-    echo "OBJCOPY=$target_objcopy"
-  fi
-  if test -n "$target_ranlib"; then
-    echo "RANLIB=$target_ranlib"
-  fi
-  if test -n "$target_strip"; then
-    echo "STRIP=$target_strip"
-  fi
-}
-
-write_container_target_makefile() {
-  echo "$1: docker-image-$container_image" >> Makefile.prereqs
-  echo "EXTRA_CFLAGS=$target_cflags"
-  if test -n "$container_cross_cc"; then
-    echo "CC=$docker_py cc --cc $container_cross_cc -i qemu/$container_image -s $source_path --"
-    echo "CCAS=$docker_py cc --cc $container_cross_cc -i qemu/$container_image -s $source_path --"
+  if test -z "$target_cc" && test -z "$target_as"; then
+    test -z "$container_image" && error_exit "Internal error: could not find cross compiler for $1?"
+    echo "$1: docker-image-$container_image" >> Makefile.prereqs
+    if test -n "$container_cross_cc"; then
+      echo "CC=$docker_py cc --cc $container_cross_cc -i qemu/$container_image -s $source_path --"
+      echo "CCAS=$docker_py cc --cc $container_cross_cc -i qemu/$container_image -s $source_path --"
+    fi
+    echo "AR=$docker_py cc --cc $container_cross_ar -i qemu/$container_image -s $source_path --"
+    echo "AS=$docker_py cc --cc $container_cross_as -i qemu/$container_image -s $source_path --"
+    echo "LD=$docker_py cc --cc $container_cross_ld -i qemu/$container_image -s $source_path --"
+    echo "NM=$docker_py cc --cc $container_cross_nm -i qemu/$container_image -s $source_path --"
+    echo "OBJCOPY=$docker_py cc --cc $container_cross_objcopy -i qemu/$container_image -s $source_path --"
+    echo "RANLIB=$docker_py cc --cc $container_cross_ranlib -i qemu/$container_image -s $source_path --"
+    echo "STRIP=$docker_py cc --cc $container_cross_strip -i qemu/$container_image -s $source_path --"
+  else
+    if test -n "$target_cc"; then
+      echo "CC=$target_cc"
+      echo "CCAS=$target_ccas"
+    fi
+    if test -n "$target_ar"; then
+      echo "AR=$target_ar"
+    fi
+    if test -n "$target_as"; then
+      echo "AS=$target_as"
+    fi
+    if test -n "$target_ld"; then
+      echo "LD=$target_ld"
+    fi
+    if test -n "$target_nm"; then
+      echo "NM=$target_nm"
+    fi
+    if test -n "$target_objcopy"; then
+      echo "OBJCOPY=$target_objcopy"
+    fi
+    if test -n "$target_ranlib"; then
+      echo "RANLIB=$target_ranlib"
+    fi
+    if test -n "$target_strip"; then
+      echo "STRIP=$target_strip"
+    fi
   fi
-  echo "AR=$docker_py cc --cc $container_cross_ar -i qemu/$container_image -s $source_path --"
-  echo "AS=$docker_py cc --cc $container_cross_as -i qemu/$container_image -s $source_path --"
-  echo "LD=$docker_py cc --cc $container_cross_ld -i qemu/$container_image -s $source_path --"
-  echo "NM=$docker_py cc --cc $container_cross_nm -i qemu/$container_image -s $source_path --"
-  echo "OBJCOPY=$docker_py cc --cc $container_cross_objcopy -i qemu/$container_image -s $source_path --"
-  echo "RANLIB=$docker_py cc --cc $container_cross_ranlib -i qemu/$container_image -s $source_path --"
-  echo "STRIP=$docker_py cc --cc $container_cross_strip -i qemu/$container_image -s $source_path --"
 }
 
-
-
 ##########################################
 # check for vfio_user_server
 
@@ -2560,15 +2558,9 @@  for target in $target_list; do
       ;;
   esac
 
-  probe_target_compiler $target
-  if test $got_cross_cc = yes; then
-      write_target_makefile >> "$config_target_mak"
-  elif test -n "$container_image"; then
-      build_static=y
-      write_container_target_makefile build-tcg-tests-$target >> "$config_target_mak"
-      got_cross_cc=yes
-  fi
-  if test $got_cross_cc = yes; then
+  if probe_target_compiler $target || test -n "$container_image"; then
+      test -n "$container_image" && build_static=y
+      write_target_makefile "build-tcg-tests-$target >> "$config_target_mak"
       mkdir -p "tests/tcg/$target"
       ln -sf "$source_path/tests/tcg/Makefile.target" "tests/tcg/$target/Makefile"
       ln -sf "../config-$target.mak" "tests/tcg/$target/config-target.mak"