qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] Re: [PATCH] build system: Further improve quiet mode


From: Jan Kiszka
Subject: [Qemu-devel] Re: [PATCH] build system: Further improve quiet mode
Date: Tue, 27 Jan 2009 10:49:16 +0100
User-agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666

Jan Kiszka wrote:
> Laurent Desnogues wrote:
>> On Mon, Jan 26, 2009 at 8:41 PM, Anthony Liguori <address@hidden> wrote:
>>> Paul Brook wrote:
>>>>> BTW, what do people think of printing CFLAGS?  I find it not all that
>>>>> exciting but if people like it, I'm happy to go with it.
>>>>>
>>>> I don't find it that interesting either. Especially given we use at least
>>>> two different sets of CFLAGS, and you really also need to know which
>>>> compiler is being invoked. You can always use V=1.
>>>>
>>> That was my thinking too.  The more noise in the build output, the less
>>> useful the quiet mode is wrt catching warnings.
>> So what about this?
>>
>>
>> Laurent
>>
>> - Make user-mode build process quieter.
>>
>> Signed-off-by: Laurent Desnogues <address@hidden>
>>
>> Index: Makefile.target
>> ===================================================================
>> --- Makefile.target  (revision 6463)
>> +++ Makefile.target  (working copy)
>> @@ -725,9 +725,9 @@
>>  endif # !CONFIG_USER_ONLY
>>  
>>  gdbstub-xml.c: $(TARGET_XML_FILES) feature_to_c.sh
>> -    rm -f $@
>> +    $(call quiet-command,rm -f $@)
>>  ifeq ($(TARGET_XML_FILES),)
>> -    echo > $@
>> +    $(call quiet-command,echo > $@)
>>  else
>>      $(SHELL) $(SRC_PATH)/feature_to_c.sh $@ $(TARGET_XML_FILES)
>>  endif
> 
> I was thinking in this direction already as well. But please go for a
> canonical output, something like
> 
>   GENERATE ppc-softmmu/gdbstub-xml.c
> 
> (Likely requires adjusting indention of the other rules.)

Or simply use

  GEN   ppc-softmmu/gdbstub-xml.c

which fits into the existing layout.

Jan

-- 
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux




reply via email to

[Prev in Thread] Current Thread [Next in Thread]