From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41478) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZaIuX-0003hX-9v for qemu-devel@nongnu.org; Fri, 11 Sep 2015 03:38:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZaIuU-0007ux-45 for qemu-devel@nongnu.org; Fri, 11 Sep 2015 03:38:17 -0400 Received: from mail-wi0-f181.google.com ([209.85.212.181]:36017) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZaIuT-0007td-Um for qemu-devel@nongnu.org; Fri, 11 Sep 2015 03:38:14 -0400 Received: by wicgb1 with SMTP id gb1so51615148wic.1 for ; Fri, 11 Sep 2015 00:38:12 -0700 (PDT) References: <1441185266-25643-1-git-send-email-jerome.forissier@linaro.org> <55F27F7B.7080104@msgid.tls.msk.ru> From: =?UTF-8?B?SsOpcsO0bWUgRm9yaXNzaWVy?= Message-ID: <55F284E1.2050200@linaro.org> Date: Fri, 11 Sep 2015 09:38:09 +0200 MIME-Version: 1.0 In-Reply-To: <55F27F7B.7080104@msgid.tls.msk.ru> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] Makefile: suppress command echoing in silent mode (make -s) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michael Tokarev , qemu-trivial@nongnu.org Cc: qemu-devel On 09/11/2015 09:15 AM, Michael Tokarev wrote: > 02.09.2015 12:14, Jerome Forissier wrote: >> Signed-off-by: Jerome Forissier > > Please Cc qemu-devel on patches. OK, sorry. I wrongly assumed that trivial patches were to be sent to qemu-trivial only to avoid "polluting" qemu-devel. Obviously I read http://wiki.qemu.org/Contribute/TrivialPatches too quickly. >> rules.mak | 13 ++++++++++++- >> 1 file changed, 12 insertions(+), 1 deletion(-) >> >> diff --git a/rules.mak b/rules.mak >> index 4551b9e..a44ba29 100644 >> --- a/rules.mak >> +++ b/rules.mak >> @@ -125,7 +125,18 @@ modules: >> %.a: >> $(call quiet-command,rm -f $@ && $(AR) rcs $@ $^," AR $(TARGET_DIR)$@") >> >> -quiet-command = $(if $(V),$1,$(if $(2),@echo $2 && $1, @$1)) >> +# Suppress command echoing in silent mode (make -s) >> +ifneq ($(filter 4.%,$(MAKE_VERSION)),) # make-4 >> +ifneq ($(filter %s ,$(firstword x$(MAKEFLAGS))),) >> +SILENT := 1 >> +endif >> +else # make-3.8x >> +ifneq ($(findstring s, $(MAKEFLAGS)),) >> +SILENT := 1 >> +endif >> +endif >> + >> +quiet-command = $(if $(V),$1,$(if $(2),$(if $(SILENT),@$1,@echo $2 && $1),@$1)) > > I'm not sure this is needed. We have V=1/V=0 way of controlling silent mode > already, Unless I'm mistaken, V does not allow silent mode. The current behavior is: - "make" prints short commands only. For instance: " CC file.o". - "make V=1" prints the full commands instead (and BTW "make V=0" does the same, which is a bit unexpected). For instance: "cc -c -o file.o file.c". But, AFAICT, there is no way to suppress all output altogether. With the proposed patch, you can use "make -s" and get *no* output unless there is an error. Silent mode is useful in continuous integration scenarios, where nobody is sitting in front of the terminal to watch the progress of the build, and it's better not to fill up the build logs with stuff unless we have an error. > why add another way, which is also twisted (depends on make version > and complicates already complex makefiles)? Yeah it's a bit unfortunate that make changed its behavior :( Thanks, -- Jerome