[Top][All Lists]

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

Re: automake-1.7b feedback (2)

From: Alexandre Duret-Lutz
Subject: Re: automake-1.7b feedback (2)
Date: Mon, 17 Nov 2003 20:08:16 +0100
User-agent: Gnus/5.1003 (Gnus v5.10.3) Emacs/21.3 (gnu/linux)

>>> "Bruno" == Bruno Haible <address@hidden> writes:

 Bruno> When I override the clean-am target, I get a warning
 Bruno> tests/ user target `clean-am' defined here...
 Bruno> automake-1.7b: ... overrides Automake target `clean-am' defined here
 Bruno> tests/ consider using clean-am-local instead of clean-am

 Bruno> However, when I follow this advice and use
 Bruno> 'clean-am-local', the generated Makefile rule for
 Bruno> 'clean-am' doesn't depend on 'clean-am-local'.  In fact,
 Bruno> nothing depends on 'clean-am-local', and so the
 Bruno> 'clean-am-local' rule is not executed when I do "make
 Bruno> clean".

Oops, thanks for catching this.  It should of course suggest you
to define clean-local, not clean-am-local.

I'm checking this in.

2003-11-17  Alexandre Duret-Lutz  <address@hidden>

        * lib/Automake/ (define): Suggest using target-local instead
        of target-am, not target-am-local.
        * tests/overrid.test: Check the diagnostic for clean-am.
        Report from Bruno Haible.

Index: lib/Automake/
RCS file: /cvs/automake/automake/lib/Automake/,v
retrieving revision 1.4
diff -u -r1.4
--- lib/Automake/        6 Sep 2003 05:36:56 -0000       1.4
+++ lib/Automake/        17 Nov 2003 19:03:40 -0000
@@ -663,7 +663,7 @@
              $noam =~ s/-am$//;
              if (exists $dependencies{"$noam-am"})
-                 $hint = "consider using $target-local instead of $target";
+                 $hint = "consider using $noam-local instead of $target";
              msg_cond_rule ('override', $cond, $target,
Index: tests/overrid.test
RCS file: /cvs/automake/automake/tests/overrid.test,v
retrieving revision 1.3
diff -u -r1.3 overrid.test
--- tests/overrid.test  3 Jul 2003 18:58:50 -0000       1.3
+++ tests/overrid.test  17 Nov 2003 19:03:40 -0000
@@ -36,6 +36,8 @@
+       :
 if COND
 ps: mine
@@ -49,6 +51,9 @@
 grep install-local stderr && exit 1 # There is no such thing as install-local
 grep installcheck-local stderr
 grep html-local stderr
+# Make sure overriding *-am targets suggest using *-local, not *-am.
+grep cleam-am-local stderr && exit 1
+grep clean-local stderr
 # Conditional overrides ought to be diagnosed, but it can't be done yet.
 # See the FIXME in rule_define.   Once this is fixed, the grep below
Alexandre Duret-Lutz

reply via email to

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