[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] docker: Build in a clean directory
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH] docker: Build in a clean directory |
Date: |
Wed, 28 Sep 2016 16:24:25 +0100 |
User-agent: |
mu4e 0.9.17; emacs 25.1.50.2 |
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