[Libreoffice] [PATCH] Reworking build.pl: What to do with the "--html" and "--interactive" parameters

[Michael Koch]

On 28.01.2011 22:53, Christian Lohmaier wrote:
> Hi Michael, *,
>
> On Fri, Jan 28, 2011 at 4:21 PM, Michael Meeks<michael.meeks at novell.com>  wrote:
>>
>> On Fri, 2011-01-28 at 15:29 +0100, Christian Lohmaier wrote:
>>> No, please don't --html in combination with --dontgraboutput is
>>> usefull for parallel builds to have the output not interwoven.
>>> (or add that feature with other means.
>>
>>         So - the requirement is that we log each build's output to somewhere,
>> as well as the console, so it can be got without being mangled up with
>> other stuff ?
>
> Having it at the console is important for tinderbox/buildbot use. the
> html switch controls the order it is being printed, not ABBBAAAB, but
> AAAABBBB - that it generates a html status page is just a side-effect
> in this usecase.
>
>>         Could we just drop that in the module's build directory ?
>
> Wenn you kill the file in the module's build directory, you kill the
> html page's "show the log" feature, as the individual module's log is
> loaded by the "master" html-page.
>
> build --html will create both the status-html (the "progress bar" as
> well as putting the module's log into<localoutput>/misc/log (IIRC).
> So it does that already. Or might be that I misundertsand completely.
>
> I don't have strong feelings about having the actual html page or not.
> When I started building, I used it quite a bit, but now I do automated
> builds, so I don't sit and wait "man, when it is finished" :-)
>
> ciao
> Christian
>

Hi Christian

I made a deeper analysis of the build.pl. --html activates verbose mode 
of "dmake" and "deliver" and --dontgraboutput calls a "cat" of a 
generated log file. This seems to be the functionality you need. I've 
now introduced a new parameter called "--verbose" which substitutes the 
functionality of "--html --dontgraboutput".

Could you please test "--verbose" in your environment to see if it works 
as expected (after this patch is pushed :-). If the tests are 
successful, I will remove all the unneeded parts. For backward 
compatibility I will link the out-dated parameters to the new --verbose 
param.

Thanks,
Michael