Re: AM_CFLAGS no longer in the manual

[Stefano Lattarini]

On 12/09/2012 05:32 PM, NightStrike wrote:
> I was looking in the manual to see what the default definition of
> AM_CFLAGS was.  It's gone.
>
Can you report the definition you used to see?  I think your repost is
spurious, because "git log -G AM_CFLAGS" tells me that, in the last
eight years, the only (two) commits to 'doc/automake.texi' that have
touched the string "AM_CFLAGS" haven't added or remove explanations.

>  Instead, all I see is this:
> 
>   AM_CFLAGS
>   This is the variable the Makefile.am author can use to pass in
>   additional C compiler flags. It is more fully documented elsewhere. In
>   some situations, this is not used, in preference to the per-executable
>   (or per-library) _CFLAGS.
>
The best fix IMHO is to just remove the "It is more fully documented
elsewhere" statement.  Done with the patch below.

> It's not more fully documented elsewhere, though.  AM_CPPFLAGS right
> about it is, however, fully documented right in this section.
>
That's because Automake adds some non-trivial stuff to the preprocessor
flags.  It doesn't do this with compiler flags.

> This should be fixed, since nowhere does the manual say that AM_CFLAGS
> is "-g -O2" by default.
> 
That's because it's not the AM_CFLAGS default.  That is the CFLAGS
determined value by Autoconf.  $(AM_CFLAGS) expands to empty by
default (hardly surprising).

Regards,
  Stefano

---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ----

>From c47c660557deec6d3faf596bebd68732d2573bcf Mon Sep 17 00:00:00 2001
Message-Id: <address@hidden>
From: Stefano Lattarini <address@hidden>
Date: Sun, 9 Dec 2012 20:00:15 +0100
Subject: [PATCH] docs: AM_CFLAGS: remove reference to non-existing "more
 detailed" desc

* doc/automake.texi (Program variables): Here.  Reported by NightStrike:
<http://lists.gnu.org/archive/html/automake/2012-12/msg00012.html>

Signed-off-by: Stefano Lattarini <address@hidden>
---
 doc/automake.texi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/doc/automake.texi b/doc/automake.texi
index 7289d85..a4e484c 100644
--- a/doc/automake.texi
+++ b/doc/automake.texi
@@ -6165,9 +6165,9 @@ same functionality.  This variable is deprecated; we 
suggest using

 @item AM_CFLAGS
 This is the variable the @file{Makefile.am} author can use to pass
-in additional C compiler flags.  It is more fully documented elsewhere.
-In some situations, this is not used, in preference to the
-per-executable (or per-library) @code{_CFLAGS}.
+in additional C compiler flags.  In some situations, this is
+not used, in preference to the per-executable (or per-library)
address@hidden

 @item COMPILE
 This is the command used to actually compile a C source file.  The
-- 
1.8.0.1.347.gf94c325