[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dis
From: |
Paolo Bonzini |
Subject: |
Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options |
Date: |
Wed, 22 Aug 2012 14:07:29 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120717 Thunderbird/14.0 |
Il 22/08/2012 14:03, Stefano Lattarini ha scritto:
>> > I'm a bit confused as to where to draw the line between Automake and GNU
>> > make...
>> >
> It depends. A rule of thumb is that, when Automake *must* process something
> at automake runtime (as is certainly the case with options!), and want to pass
> it in the generated Makefiles as well (for possible further processing there),
> it can as well "massage" it in a form that make its use in those Makefiles
> as simple and natural as possible.
>
> For example, take a look at this fragment in '&generate_makefile' :
>
> # Must come after invocation of handle_programs, handle_libraries and
> # handle_ltlibraries, so that %known_programs and %known_libraries are
> # up-to-date.
> define_variable 'am.all-progs', INTERNAL, sort keys %known_programs;
> define_variable 'am.all-libs', INTERNAL, sort keys %known_libraries;
> define_variable 'am.all-ltlibs', INTERNAL, sort keys %known_ltlibraries;
>
> Those variables could as well be defined purely at make runtime, by
> peeking into $(.VARIABLES) and munging it properly. But since we
> currently *need* '%known_programs' etc. at automake runtime anyway,
> we make life simpler and pass on that knowledge directly into the
> generated Makefiles.
True, but in the make dist case, automake has otherwise no business in
parsing the dist-format options. All it has to do is to raise a
warning, it doesn't record them in a data structure that is specific to
dist formats.
Paolo
- [Automake-NG] [PATCH v2 0/2] dist: add back support for obsolete dist-* options, Paolo Bonzini, 2012/08/22
- [Automake-NG] [PATCH v2 1/2] var: format all options in the Makefile.in output, Paolo Bonzini, 2012/08/22
- [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Paolo Bonzini, 2012/08/22
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Stefano Lattarini, 2012/08/22
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Paolo Bonzini, 2012/08/22
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Stefano Lattarini, 2012/08/22
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options,
Paolo Bonzini <=
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Stefano Lattarini, 2012/08/22
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Paolo Bonzini, 2012/08/22
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Stefano Lattarini, 2012/08/22
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Paolo Bonzini, 2012/08/22
- Re: [Automake-NG] [PATCH v2 2/2] dist: add back support for obsolete dist-* options, Stefano Lattarini, 2012/08/22