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

[Michael Koch]

Sorry, I noticed a mistake in my last patch. Here is the amended version.

Michael

On 02.02.2011 19:23, Michael Koch wrote:
> 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
>
>
>
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice