Re: Patch: couple of doc updates

[Tom Tromey]

>>>>> "adl" == Alexandre Duret-Lutz <address@hidden> writes:

adl> I'm committing this patch.  With all the FIXMEs.

Thanks.  I started reading the patch but ran out of time.  I meant to
tell you to check it in but never got around to it.

adl> | Automake tries to group comments with adjoining targets and macro
adl> | definitions in an intelligent way.
adl> |address@hidden FIXME: What does this imply practically?

Hopefully it is slightly easier to read the Makefile.in.

adl> |address@hidden FIXME: This is broken today: &am_primary_prefix will allow 
data_PROGRAMS
adl> |address@hidden because $datadir is defined before Makefile.am is parsed.  
This
adl> |address@hidden means that passing the list of allowed directory suffix to
adl> |address@hidden &am_install_var or &am_primary_prefix is absolutely 
useless (except
adl> |address@hidden for 'noinst', 'check' and friends).

Please submit a PR for this.

adl> |address@hidden FIXME: maybe Automake could bark if a developer forces a 
user variable
adl> |address@hidden from Makefile.am.

This sounds like a good idea to me.  I see this mistake made quite a
bit.  OTOH I wonder if it will just generate a lot of complaints.

adl> |address@hidden FIXME: As all the Hello World excerpts in this manual, this
adl> |address@hidden shows deprecated features (here: $(INCLUDES)).

The examples in the manual haven't been updated in a long time.
The bigger the example, the less up-to-date it probably is.

adl> |address@hidden FIXME: This seems to be a left-over from Automake 1.4, 
unused today.

Yes, we ought to deprecate this.  It used to be used by `make dist'.
Could you submit a PR for this too?

adl> |address@hidden FIXME: Consider not documenting these internal macros.
adl> |address@hidden FIXME: Consider not documenting this internal macro.
adl> |address@hidden FIXME: Consider not documenting this internal macro.

Maybe we could mention them in a separate node with a note saying
"don't use these, and if you do you're screwed in the next release".

Thanks again for writing this patch.

Tom