m4-patches
[Top][All Lists]
Advanced

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

Re: use build-aux


From: Eric Blake
Subject: Re: use build-aux
Date: Tue, 06 Nov 2007 06:15:48 -0700
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.6) Gecko/20070728 Thunderbird/2.0.0.6 Mnenhy/0.7.5.666

According to Ralf Wildenhues on 11/6/2007 4:31 AM:
> You need to ensure build-aux/mkstamp exists at the end of the bootstrap.
> You could copy it from ltdl/config/.

Or is there a better way to make libtool share build-aux, rather than
creating its own subdirectory?  Perhaps using the libtoolize
--nonrecursive or --recursive option?

> 
> FWIW I think libtool should move to using build-aux only after moving to
> git, thus after 2.2.

Was this sentence meant in the context of using build-aux within the
libtool source tree, or the context of making libtoolize use build-aux of
the project where libtool is copied to?

> 
> BTW, HEAD m4 needs the same bash redirected compound command workaround
> for package.m4 as Autoconf.

Thanks.  Applying:

From: Eric Blake <address@hidden>
Date: Tue, 6 Nov 2007 06:14:37 -0700
Subject: [PATCH] * Makefile.am (tests/package.m4): Work around bash bug.

Reported by Ralf Wildenhues.

Signed-off-by: Eric Blake <address@hidden>
---
 ChangeLog   |    5 +++++
 Makefile.am |    4 +++-
 2 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 1526687..0a986c6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2007-11-06  Eric Blake  <address@hidden>
+
+       * Makefile.am (tests/package.m4): Work around bash bug.
+       Reported by Ralf Wildenhues.
+
 2007-11-05  Eric Blake  <address@hidden>

        For consistency with other GNU projects, use build-aux directory.
diff --git a/Makefile.am b/Makefile.am
index a2e67d9..20fc3c5 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -394,8 +394,10 @@ $(TESTSUITE): tests/package.m4 $(TESTSUITE_AT)
        $(AUTOTEST) -I '$(srcdir)/tests' -o address@hidden address@hidden
        mv address@hidden $@

+# The leading :; works around a bug in {}>f in some bash versions.
 $(srcdir)/tests/package.m4: configure.ac
-       {                                       \
+       :; \
+       { \
          echo '# Signature of the current package.'; \
          echo 'm4_define([AT_PACKAGE_NAME],      address@hidden@])'; \
          echo 'm4_define([AT_PACKAGE_TARNAME],   address@hidden@])'; \
-- 
1.5.3.2

-- 
Don't work too hard, make some time for fun as well!

Eric Blake             address@hidden




reply via email to

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