[Top][All Lists]

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

reply via email to

[Prev in Thread] Current Thread [Next in Thread]