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
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 |