[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] docker: Don't allocate tty unless DEBUG=1
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH] docker: Don't allocate tty unless DEBUG=1 |
Date: |
Fri, 13 Oct 2017 10:35:04 +0100 |
User-agent: |
mu4e 0.9.19; emacs 26.0.60 |
Fam Zheng <address@hidden> writes:
> The existence of tty in the container seems to urge gcc into colorize
> the output, 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>
I certainly improves my Emacs compilation-mode buffer when applied:
Reviewed-by: Alex Bennée <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)" \
--
Alex Bennée