From: "Alex Bennée" <alex.bennee@linaro.org>
To: Fam Zheng <famz@redhat.com>
Cc: kwolf@redhat.com, peter.maydell@linaro.org, sw@weilnetz.de,
qemu-devel@nongnu.org, stefanha@redhat.com,
Paolo Bonzini <pbonzini@redhat.com>,
jsnow@redhat.com, david@gibson.dropbear.id.au
Subject: Re: [Qemu-devel] [PATCH v4 12/13] docker: Add EXTRA_CONFIGURE_OPTS
Date: Thu, 31 Mar 2016 14:20:33 +0100 [thread overview]
Message-ID: <871t6qhjha.fsf@linaro.org> (raw)
In-Reply-To: <1458196505-5473-13-git-send-email-famz@redhat.com>
Fam Zheng <famz@redhat.com> writes:
> Whatever passed in this variable will be appended to all
> configure commands.
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
>
> Signed-off-by: Fam Zheng <famz@redhat.com>
> ---
> tests/docker/Makefile.include | 3 +++
> tests/docker/common.rc | 1 +
> 2 files changed, 4 insertions(+)
>
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index 033809a..4270a44 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -86,6 +86,8 @@ docker:
> @echo
> @echo 'Special variables:'
> @echo ' TARGET_LIST=a,b,c Override target list in builds.'
> + @echo ' EXTRA_CONFIGURE_OPTS="..."'
> + @echo ' Extra configure options.'
> @echo ' IMAGES="a b c ..": Filters which images to build or run.'
> @echo ' TESTS="x y z .." Filters which tests to run (for docker-test).'
> @echo ' J=[0..9]* Overrides the -jN parameter for make commands'
> @@ -106,6 +108,7 @@ docker-run-%: docker-qemu-src
> --privileged -t \
> $(if $(DEBUG),-i,--net=none) \
> -e TARGET_LIST=$(TARGET_LIST) \
> + -e EXTRA_CONFIGURE_OPTS=$(EXTRA_CONFIGURE_OPTS) \
> -e V=$V -e J=$J -e DEBUG=$(DEBUG)\
> -e CCACHE_DIR=/var/tmp/ccache \
> -v $$(realpath $(SRC_COPY)):/var/tmp/qemu:ro \
> diff --git a/tests/docker/common.rc b/tests/docker/common.rc
> index 74b89d6..c493eeb 100755
> --- a/tests/docker/common.rc
> +++ b/tests/docker/common.rc
> @@ -26,6 +26,7 @@ build_qemu()
> $QEMU_SRC/configure \
> --target-list="${TARGET_LIST}" \
> --prefix="$PWD/install" \
> + $EXTRA_CONFIGURE_OPTS \
> "$@"
> make $MAKEFLAGS
> }
--
Alex Bennée
next prev parent reply other threads:[~2016-03-31 13:20 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-17 6:34 [Qemu-devel] [PATCH v4 00/13] tests: Introducing docker tests Fam Zheng
2016-03-17 6:34 ` [Qemu-devel] [PATCH v4 01/13] tests: Add utilities for docker testing Fam Zheng
2016-03-31 9:47 ` Alex Bennée
2016-03-17 6:34 ` [Qemu-devel] [PATCH v4 02/13] Makefile: Rules " Fam Zheng
2016-03-31 9:52 ` Alex Bennée
2016-03-31 10:32 ` Alex Bennée
2016-03-17 6:34 ` [Qemu-devel] [PATCH v4 03/13] docker: Add images Fam Zheng
2016-03-17 6:34 ` [Qemu-devel] [PATCH v4 04/13] docker: Add test runner Fam Zheng
2016-03-17 6:34 ` [Qemu-devel] [PATCH v4 05/13] docker: Add common.rc Fam Zheng
2016-03-17 6:34 ` [Qemu-devel] [PATCH v4 06/13] docker: Add quick test Fam Zheng
2016-03-17 6:34 ` [Qemu-devel] [PATCH v4 07/13] docker: Add full test Fam Zheng
2016-03-31 13:26 ` Alex Bennée
2016-03-17 6:35 ` [Qemu-devel] [PATCH v4 08/13] docker: Add clang test Fam Zheng
2016-03-31 13:29 ` Alex Bennée
2016-03-17 6:35 ` [Qemu-devel] [PATCH v4 09/13] docker: Add mingw test Fam Zheng
2016-03-17 6:35 ` [Qemu-devel] [PATCH v4 10/13] docker: Add travis tool Fam Zheng
2016-03-31 13:14 ` Alex Bennée
2016-03-17 6:35 ` [Qemu-devel] [PATCH v4 11/13] docs: Add text for tests/docker in build-system.txt Fam Zheng
2016-03-17 6:35 ` [Qemu-devel] [PATCH v4 12/13] docker: Add EXTRA_CONFIGURE_OPTS Fam Zheng
2016-03-31 13:20 ` Alex Bennée [this message]
2016-03-17 6:35 ` [Qemu-devel] [PATCH v4 13/13] MAINTAINERS: Add tests/docker Fam Zheng
2016-03-28 2:37 ` [Qemu-devel] [PATCH v4 00/13] tests: Introducing docker tests Fam Zheng
2016-03-28 11:50 ` Alex Bennée
2016-05-03 8:20 ` Alex Bennée
2016-05-03 9:36 ` Fam Zheng
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=871t6qhjha.fsf@linaro.org \
--to=alex.bennee@linaro.org \
--cc=david@gibson.dropbear.id.au \
--cc=famz@redhat.com \
--cc=jsnow@redhat.com \
--cc=kwolf@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
--cc=sw@weilnetz.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.