automake-commit
[Top][All Lists]
Advanced

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

[Automake-commit] [SCM] GNU Automake branch, ng/master, updated. v1.12.1


From: Stefano Lattarini
Subject: [Automake-commit] [SCM] GNU Automake branch, ng/master, updated. v1.12.1-293-g165c66c
Date: Sat, 02 Jun 2012 11:57:38 +0000

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU Automake".

http://git.sv.gnu.org/gitweb/?p=automake.git;a=commitdiff;h=165c66c01fc0db7654de590a46099ed915c3d8a6

The branch, ng/master has been updated
       via  165c66c01fc0db7654de590a46099ed915c3d8a6 (commit)
      from  a5d053bf0ad047bb461edbaf4cea6d403930f17b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 165c66c01fc0db7654de590a46099ed915c3d8a6
Author: Stefano Lattarini <address@hidden>
Date:   Sat Jun 2 13:55:15 2012 +0200

    texinfo: fix an imprecise comment
    
    In GNU make, '$^' used in a recipe contains every dependency for the
    target, even those not declared when the recipe is read; for example,
    on:
    
        all: foo1; @echo $^
        all: foo2
    
    "make all" would output "foo1 foo2".  This is not Automake's fault,
    as some comments incorrectly stated.  See also:
    <http://lists.gnu.org/archive/html/help-make/2012-05/msg00029.html>
    
    * lib/am/texinfos.am: Fix those incorrect comments, and document the
    GNU make behaviour described above.
    
    Signed-off-by: Stefano Lattarini <address@hidden>

-----------------------------------------------------------------------

Summary of changes:
 lib/am/texinfos.am |   18 ++++++++++--------
 1 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/lib/am/texinfos.am b/lib/am/texinfos.am
index afe08a1..fc3981a 100644
--- a/lib/am/texinfos.am
+++ b/lib/am/texinfos.am
@@ -136,14 +136,16 @@ if %?LOCAL-TEXIS%
 
 include inst-vars.am
 
-## FIXME: Extra indirection required because Automake proprocessing groups
-## FIXME: dependencies for a given '*-am' together, so that we might end up
-## FIXME: in our genereated Makefile with something like:
-## FIXME:
-## FIXME:   install-pdf-am: $(PDFS) install-pdf-local
-## FIXME:           ... [RECIPE] ...
-## FIXME: 
-## FIXME: which would cause a huge mess with our usages of '$^' in the recipes.
+# In GNU make, '$^' used in a recipe contains every dependency for the
+# target, even those not declared when the recipe is read; for example,
+# on:
+#    all: foo1; @echo $^
+#    all: foo2
+# "make all" would output "foo1 foo2".  In our usage, a dependecy like
+# "install-pdf-am: install-pdf-local" (that is automatically output by
+# Automake-NG if the 'install-pdf-local' target is declared) would make
+# '$^' unusable as a pure list of PDF target files in the recipe of
+# 'install-pdf-am'.  So we need the following indirections.
 
 install-dvi-am:  am--install-dvi
 install-ps-am:   am--install-ps


hooks/post-receive
-- 
GNU Automake



reply via email to

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