Re: [Qemu-devel] [PATCH] docker: Build in a clean directory

[Alex Bennée]

Fam Zheng <address@hidden> writes:

> Currently we configure and build under "$QEMU_SRC/tests/docker" which is
> dubious, create a fixed directory (to be friendly to ccache) and change
> to there before calling build_qemu.
>
> Signed-off-by: Fam Zheng <address@hidden>
> ---
>  tests/docker/common.rc  | 3 +++
>  tests/docker/test-clang | 2 ++
>  tests/docker/test-full  | 2 ++
>  tests/docker/test-mingw | 2 ++
>  tests/docker/test-quick | 2 ++
>  5 files changed, 11 insertions(+)
>
> diff --git a/tests/docker/common.rc b/tests/docker/common.rc
> index 510a3ad..21657e8 100755
> --- a/tests/docker/common.rc
> +++ b/tests/docker/common.rc
> @@ -11,6 +11,9 @@
>  # or (at your option) any later version. See the COPYING file in
>  # the top-level directory.
>
> +BUILD_DIR=/var/tmp/qemu-build
> +mkdir $BUILD_DIR

Is that safe on multi-user machines?

Also should we honour $TMPDIR if set?

> +
>  requires()
>  {
>      for c in $@; do
> diff --git a/tests/docker/test-clang b/tests/docker/test-clang
> index 60e4e97..16485e6 100755
> --- a/tests/docker/test-clang
> +++ b/tests/docker/test-clang
> @@ -15,6 +15,8 @@
>
>  requires clang
>
> +cd "$BUILD_DIR"
> +
>  OPTS="--enable-debug --cxx=clang++ --cc=clang --host-cc=clang"
>  # -fsanitize=undefined is broken on Fedora 23, skip it for now
>  # See also: https://bugzilla.redhat.com/show_bug.cgi?id=1263834
> diff --git a/tests/docker/test-full b/tests/docker/test-full
> index fd9b798..05f0d49 100755
> --- a/tests/docker/test-full
> +++ b/tests/docker/test-full
> @@ -13,5 +13,7 @@
>
>  . common.rc
>
> +cd "$BUILD_DIR"
> +
>  build_qemu
>  make check $MAKEFLAGS
> diff --git a/tests/docker/test-mingw b/tests/docker/test-mingw
> index c03757a..3396876 100755
> --- a/tests/docker/test-mingw
> +++ b/tests/docker/test-mingw
> @@ -15,6 +15,8 @@
>
>  requires mingw dtc
>
> +cd "$BUILD_DIR"
> +
>  for prefix in x86_64-w64-mingw32- i686-w64-mingw32-; do
>      TARGET_LIST=x86_64-softmmu,aarch64-softmmu \
>          build_qemu --cross-prefix=$prefix \
> diff --git a/tests/docker/test-quick b/tests/docker/test-quick
> index 7885dfa..c465dc0 100755
> --- a/tests/docker/test-quick
> +++ b/tests/docker/test-quick
> @@ -13,6 +13,8 @@
>
>  . common.rc
>
> +cd "$BUILD_DIR"
> +
>  DEF_TARGET_LIST="x86_64-softmmu,aarch64-softmmu"
>  TARGET_LIST=${TARGET_LIST:-$DEF_TARGET_LIST} \
>  build_qemu


--
Alex Bennée