On 06/30/2012 09:33 PM, Dave Hart wrote:
> On Sat, Jun 30, 2012 at 13:16 UTC, Stefano Lattarini wrote:
>
>> Ah, but the nice thing about this new feature is that you don't need
>> to define a 'foo-local' target in each of your Makefiles in order to
>> support a recursive 'foo' target! You only have to define 'foo-local'
>> where it has actually something to do. This is not clear from your
>> formulation IMO. Do you have an alternative formulation that takes
>> my clarification into account?
>
> I tried to communicate that, but we could add "Automake generates
> corresponding stub @code{-local} targets when Makefile.am does not
> provide such target."
>
+1. I will adjust the documentation along this line.
>> In light of my explanation above, I now think this example should be
>> extended as follows:
>>
>> % @kbd{cat Makefile.am}
>> SUBDIRS = sub
>> foo-local:
>> @echo This will be run by "make foo".
>> % @kbd{cat sub/Makefile.am}
>> SUBDIRS = src
>> % @kbd{cat sub/src/Makefile.am}
>> foo-local:
>> @echo This too will be run by a "make foo" issued either in
>> @echo the 'sub/src' directory, in the 'sub/' directory, or
>> @echo in the top-level directory.
>
> I agree highlighting the intermediate directory doesn't need to
> provide a foo-local target improves the example.
>
Good!
I will push the patch by tomorrow if there is no further feedback.
Thanks,
Stefano
