[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Some serious issues with the new -O option
From: |
Stefano Lattarini |
Subject: |
Re: Some serious issues with the new -O option |
Date: |
Wed, 01 May 2013 14:16:16 +0200 |
Hi Paul.
On 05/01/2013 02:04 PM, Paul Smith wrote:
> On Tue, 2013-04-30 at 10:39 -0400, Paul Smith wrote:
>> On Tue, 2013-04-30 at 16:04 +0200, Stefano Lattarini wrote:
>>> On 04/30/2013 03:37 PM, Paul Smith wrote:
>>>> Just to be clear, you're saying that the testsuite runs as one long
>>>> operation, updating one target, and the recipe invokes one test script,
>>>> right?
>>>>
>>> No; the testsuite runs as a recursive make invocation (yes, this is
>>> sadly truly needed in order to support all the features offered by the
>>> Automake parallel testsuite harness --- sorry), but each test script
>>> (and there can be hundreds of them, as is the case for GNU coreutils
>>> or GNU automake itself) is run as a separate target, explicit for
>>> tests which have no extension and pattern-based for tests that have an
>>> extension.
>>
>> This should work very well with -Otarget then, except for the
>> colorization/highlighting issue... once it works as expected. I'll look
>> into this issue later and I would be interested to see your experience
>> with it once it's resolved.
>
> OK, I found this bug. Definitely make recursion was not being handled
> properly with -Otarget and -Ojob in some situations; this broke as a
> side effect of my cleanup to reuse the same temporary file for the
> entire target, regardless of the output mode.
>
> This should be fixed now. Those who use recursive makefiles and were
> seeing annoying delays in output with -O, please try again with the
> latest commit and see if it works any better for you now.
>
I can confirm the problem has disappeared for me. Thanks!
(The issue with extra "Entering/Leaving directory" messages is still
present though; I hope it can be fixed before the release ...).
Best regards,
Stefano
- Re: Some serious issues with the new -O option, Paul Smith, 2013/05/01
- Re: Some serious issues with the new -O option,
Stefano Lattarini <=
- Re: Some serious issues with the new -O option, Eli Zaretskii, 2013/05/01
- Re: Some serious issues with the new -O option, Stefano Lattarini, 2013/05/01
- Re: Some serious issues with the new -O option, Paul Smith, 2013/05/01
- Re: Some serious issues with the new -O option, Eli Zaretskii, 2013/05/01
- Re: Some serious issues with the new -O option, Paul Smith, 2013/05/01
- Re: Some serious issues with the new -O option, Eli Zaretskii, 2013/05/01
- Re: Some serious issues with the new -O option, Tim Murphy, 2013/05/02
- Re: Some serious issues with the new -O option, Paul Smith, 2013/05/02
- Re: Some serious issues with the new -O option, Eli Zaretskii, 2013/05/02
- Re: Some serious issues with the new -O option, Paul Smith, 2013/05/02