diff options
author | Fam Zheng <famz@redhat.com> | 2017-10-13 09:19:54 +0800 |
---|---|---|
committer | Fam Zheng <famz@redhat.com> | 2017-10-20 14:28:50 +0800 |
commit | 4b2c6bc563c13f242d92fe4f8f31d22b31788858 (patch) | |
tree | 3e1799bc3b3657a2f618219339034b7056a2074e /tests/docker | |
parent | 063833a6ec2a6747e27c5f9866bb44c7e8de1265 (diff) | |
download | qemu-4b2c6bc563c13f242d92fe4f8f31d22b31788858.zip qemu-4b2c6bc563c13f242d92fe4f8f31d22b31788858.tar.gz qemu-4b2c6bc563c13f242d92fe4f8f31d22b31788858.tar.bz2 |
docker: Don't allocate tty unless DEBUG=1
The existence of tty in the container seems to urge gcc into colorizing
the errors, but the escape chars will clutter the report once turned
into email replies on patchew. Move -t to debug mode.
Reported-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
Message-Id: <20171013011954.9975-1-famz@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Fam Zheng <famz@redhat.com>
Diffstat (limited to 'tests/docker')
-rw-r--r-- | tests/docker/Makefile.include | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include index 6f9ea19..ab939f2 100644 --- a/tests/docker/Makefile.include +++ b/tests/docker/Makefile.include @@ -134,10 +134,10 @@ docker-run: docker-qemu-src " COPYING $(EXECUTABLE) to $(IMAGE)")) $(call quiet-command, \ $(SRC_PATH)/tests/docker/docker.py run \ - $(if $(NOUSER),,-u $(shell id -u)) -t \ + $(if $(NOUSER),,-u $(shell id -u)) \ --security-opt seccomp=unconfined \ $(if $V,,--rm) \ - $(if $(DEBUG),-i,) \ + $(if $(DEBUG),-ti,) \ $(if $(NETWORK),$(if $(subst $(NETWORK),,1),--net=$(NETWORK)),--net=none) \ -e TARGET_LIST=$(TARGET_LIST) \ -e EXTRA_CONFIGURE_OPTS="$(EXTRA_CONFIGURE_OPTS)" \ |