Re: Lower msvc build verbosity level

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Magnus Hagander <magnus(at)hagander(dot)net>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Christian Ullrich <chris(at)chrullrich(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Lower msvc build verbosity level
Date: 2016-04-08 12:44:55
Message-ID: CAB7nPqRD79ABj5hSoXTtRHm_=KZN23sVW_FOZzm=3Lm-cvmNdQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Apr 8, 2016 at 9:29 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> On Fri, Apr 8, 2016 at 2:25 PM, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>> On 04/08/2016 06:19 AM, Christian Ullrich wrote:
>>> could we perhaps lower the verbosity level of the msvc build (in
>>> src/tools/msvc/build.pl) from "detailed" to "normal"? In my experiment, this
>>> reduces the size of the build log by 96.4 percent (from 12.5 MiB to 438
>>> KiB), or if the log is not redirected, it shortens the build time by 45
>>> percent, from just under 6 minutes to 3.
>>>
>>> The lines that would not be logged anymore are mostly "Task [or Target]
>>> so-and-so skipped, due to false condition". All the compiler, linker, etc.
>>> command lines are still there.
>>>
>>> Also, with the "detailed" logs, loading the buildfarm status page for a
>>> failure in the Make stage is really painful.
>>>
>>
>> Yes! I was just wondering yesterday how to do this. If nobody objects I'm
>> going to apply and backpatch this quite soon. This will also have beneficial
>> effects on the buildfarm server.
>
> +1 for doing that. It seems easy enough to manually turn it on if it turns
> out to be necessary to debug some specific issue in the future.

Please, yes. The current output is too verbose.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Anastasia Lubennikova 2016-04-08 12:45:49 Re: WIP: Covering + unique indexes.
Previous Message Aleksander Alekseev 2016-04-08 12:32:17 Small patch for snapmgr.c