[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more proces
From: |
Dave Hart |
Subject: |
Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime |
Date: |
Fri, 15 Jun 2012 13:28:34 +0000 |
On Fri, Jun 15, 2012 at 1:23 PM, Stefano Lattarini
<address@hidden> wrote:
> On 06/15/2012 03:12 PM, Dave Hart wrote:
>> On Thu, Jun 14, 2012 at 10:05 PM, Stefano Lattarini
>> <address@hidden> wrote:
>>> * lib/am/header-vars.am: Sanitize $(SUBDIRS), by setting it to the
>>> empty value if it is not set, but not from the Makefile not any
>>> command-line override.
>>
>> I'm tied up in "nots" here. The part after "but" confuses me. I
>> didn't spend time reading the diff trying to understand what was
>> meant, I'm hoping you'll see the problem with the summary and clarify.
>>
> Oh damn, I'd just pushed before reading your message :-/
>
> And sadly you are right, and the wording is botched: SUBDIRS is now being
> set to the empty value if it *is* set, but not from the Makefile not any
> command-line override.
I have the same problem understanding that last sentence as I do with
the earlier summary: I don't understand what you're trying to say
after "but".
> Oh well, what's done it done.
Is there no reasonable way to improve summary text after the initial push?
Cheers,
Dave Hart
- [Automake-NG] [PATCH 0/2] all, check, install: move more processing at make runtime, Stefano Lattarini, 2012/06/14
- [Automake-NG] [PATCH 1/2] [ng] built sources: enhance testsuite coverage, Stefano Lattarini, 2012/06/14
- [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime, Stefano Lattarini, 2012/06/14
- Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime, Akim Demaille, 2012/06/15
- Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime, Stefano Lattarini, 2012/06/15
- Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime, Akim Demaille, 2012/06/15
- Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime, Stefano Lattarini, 2012/06/15
- Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime, Dave Hart, 2012/06/15
- Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime, Stefano Lattarini, 2012/06/15
- Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime,
Dave Hart <=
- Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime, Eric Blake, 2012/06/15