[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 1/5] docker: Don't allocate tty unless DEBUG=1
From: |
Fam Zheng |
Subject: |
[Qemu-devel] [PULL 1/5] docker: Don't allocate tty unless DEBUG=1 |
Date: |
Fri, 20 Oct 2017 14:32:15 +0800 |
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 <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>
---
tests/docker/Makefile.include | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
index 6f9ea196a7..ab939f2bec 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)" \
--
2.13.5
- [Qemu-devel] [PULL 0/5] Docker patches, Fam Zheng, 2017/10/20
- [Qemu-devel] [PULL 2/5] docker: cleanup temp directory after test, Fam Zheng, 2017/10/20
- [Qemu-devel] [PULL 1/5] docker: Don't allocate tty unless DEBUG=1,
Fam Zheng <=
- [Qemu-devel] [PULL 3/5] docker: allow running from srcdir != builddir build, Fam Zheng, 2017/10/20
- [Qemu-devel] [PULL 4/5] docker: fix out-of-tree 'make address@hidden', Fam Zheng, 2017/10/20
- [Qemu-devel] [PULL 5/5] docker: Fix PATH for ccache, Fam Zheng, 2017/10/20
- Re: [Qemu-devel] [PULL 0/5] Docker patches, Peter Maydell, 2017/10/20