automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Patch: FYI: PR 311


From: Tom Tromey
Subject: Patch: FYI: PR 311
Date: 29 Mar 2002 17:32:03 -0700

I'm checking this in.
This fixes PR 311.

Tom

Index: ChangeLog
from  Tom Tromey  <address@hidden>
        * automake.in (check_typos): Examine AM_LDFLAGS.
        Fixes PR automake/311.

Index: automake.in
===================================================================
RCS file: /cvs/automake/automake/automake.in,v
retrieving revision 1.1286
diff -u -r1.1286 automake.in
--- automake.in 2002/03/30 00:26:05 1.1286
+++ automake.in 2002/03/30 00:30:53
@@ -3073,6 +3073,9 @@
 # EXTRA_ variables don't contain configure substitutions.
 sub check_typos ()
 {
+    # It is ok if the user sets this particular variable.
+    &examine_variable ('AM_LDFLAGS');
+
     foreach my $varname (keys %var_value)
     {
        foreach my $primary ('_SOURCES', '_LIBADD', '_LDADD', '_LDFLAGS',
@@ -3080,11 +3083,11 @@
        {
            macro_error ($varname,
                         "invalid unused variable name: `$varname'")
-           # Note that a configure variable is always legitimate.
-           # It is natural to name such variables after the
-           # primary, so we explicitly allow it.
-           if $varname =~ /$primary$/ && ! $content_seen{$varname}
-               && ! exists $configure_vars{$varname};
+               # Note that a configure variable is always legitimate.
+               # It is natural to name such variables after the
+               # primary, so we explicitly allow it.
+               if $varname =~ /$primary$/ && ! $content_seen{$varname}
+                  && ! exists $configure_vars{$varname};
        }
     }
 }



reply via email to

[Prev in Thread] Current Thread [Next in Thread]