[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: minor improvements for {...} problem; also, prefer && to ;
From: |
Ralf Wildenhues |
Subject: |
Re: minor improvements for {...} problem; also, prefer && to ; |
Date: |
Fri, 9 Nov 2007 19:55:37 +0100 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
Hello Paul,
* Paul Eggert wrote on Fri, Nov 09, 2007 at 09:47:06AM CET:
> I installed this:
>
> 2007-11-09 Paul Eggert <address@hidden>
>
> * doc/autoconf.texi (Limitations of Builtins): Document problem
> with { ... } a bit more clearly. Suggest ":;{" as a shorthand
> for the workaround.
> * lib/m4sugar/Makefile.am (version.m4): Detect 'echo' failure.
> Use ":;{" shorthand.
> * tests/Makefile.am ($(srcdir)/package.m4): Likewise.
Could you git-push, please, so that I can apply this followup patch?
Thank you,
Ralf
* doc/autoconf.texi (Making testsuite Scripts): Document
":;{" shorthand as in previous patch.
diff --git a/doc/autoconf.texi b/doc/autoconf.texi
index 421056e..8c3a17d 100644
--- a/doc/autoconf.texi
+++ b/doc/autoconf.texi
@@ -20231,13 +20231,14 @@ we suggest that you also define
@code{AT_PACKAGE_NAME},
suggest the following makefile excerpt:
@smallexample
+# The `:;' works around a Bash 3.2 bug when the output is not writeable.
$(srcdir)/package.m4: $(top_srcdir)/configure.ac
- @{ \
- echo '# Signature of the current package.'; \
- echo 'm4_define([AT_PACKAGE_NAME], [@@PACKAGE_NAME@@])'; \
- echo 'm4_define([AT_PACKAGE_TARNAME], [@@PACKAGE_TARNAME@@])'; \
- echo 'm4_define([AT_PACKAGE_VERSION], [@@PACKAGE_VERSION@@])'; \
- echo 'm4_define([AT_PACKAGE_STRING], [@@PACKAGE_STRING@@])'; \
+ :;@{ \
+ echo '# Signature of the current package.' && \
+ echo 'm4_define([AT_PACKAGE_NAME], [@@PACKAGE_NAME@@])' && \
+ echo 'm4_define([AT_PACKAGE_TARNAME], [@@PACKAGE_TARNAME@@])' && \
+ echo 'm4_define([AT_PACKAGE_VERSION], [@@PACKAGE_VERSION@@])' && \
+ echo 'm4_define([AT_PACKAGE_STRING], [@@PACKAGE_STRING@@])' && \
echo 'm4_define([AT_PACKAGE_BUGREPORT], [@@PACKAGE_BUGREPORT@@])'; \
@} >'$(srcdir)/package.m4'
@end smallexample