* [patch] 1.6.4: aclocal 1.8 compatibility
@ 2004-07-30 12:16 Maciej W. Rozycki
2004-07-31 1:57 ` Kevin Ryde
0 siblings, 1 reply; 4+ messages in thread
From: Maciej W. Rozycki @ 2004-07-30 12:16 UTC (permalink / raw)
Hello,
Since automake 1.8 aclocal requires macro names being defined by AC_DEFUN
to be quoted. Here is a patch that adds missing quotations.
2004-07-30 Maciej W. Rozycki <macro@linux-mips.org>
* acinclude.m4: Quote names of macros to be defined by AC_DEFUN
throughout.
* configure.in: Likewise.
* aclocal.m4: Regenerate.
* configure: Regenerate.
Please apply.
Maciej
guile-1.6.4-am18.patch
diff -up --recursive --new-file guile-1.6.4.macro/acinclude.m4 guile-1.6.4/acinclude.m4
--- guile-1.6.4.macro/acinclude.m4 2002-12-07 22:41:28.000000000 +0000
+++ guile-1.6.4/acinclude.m4 2004-07-26 21:21:30.000000000 +0000
@@ -1,7 +1,7 @@
dnl On the NeXT, #including <utime.h> doesn't give you a definition for
dnl struct utime, unless you #define _POSIX_SOURCE.
-AC_DEFUN(GUILE_STRUCT_UTIMBUF, [
+AC_DEFUN([GUILE_STRUCT_UTIMBUF], [
AC_CACHE_CHECK([whether we need POSIX to get struct utimbuf],
guile_cv_struct_utimbuf_needs_posix,
[AC_TRY_CPP([
@@ -71,7 +71,7 @@ dnl with other parameters, such as libra
dnl
dnl GUILE_NAMED_CHECK_FUNC(FUNCTION, TESTNAME,
dnl [ACTION-IF-FOUND [, ACTION-IF-NOT-FOUND]])
-AC_DEFUN(GUILE_NAMED_CHECK_FUNC,
+AC_DEFUN([GUILE_NAMED_CHECK_FUNC],
[AC_MSG_CHECKING([for $1])
AC_CACHE_VAL(ac_cv_func_$1_$2,
[AC_TRY_LINK(
diff -up --recursive --new-file guile-1.6.4.macro/configure.in guile-1.6.4/configure.in
--- guile-1.6.4.macro/configure.in 2002-12-08 01:48:52.000000000 +0000
+++ guile-1.6.4/configure.in 2004-07-26 21:25:11.000000000 +0000
@@ -279,7 +279,7 @@ AC_CHECK_LIB(crypt, crypt)
### Check for a declaration of FUNCTION in HEADERFILE; if it is
### not there, #define MISSING_FUNCTION_DECL.
-AC_DEFUN(GUILE_FUNC_DECLARED, [
+AC_DEFUN([GUILE_FUNC_DECLARED], [
AC_CACHE_CHECK(for $1 declaration, guile_cv_func_$1_declared,
AC_EGREP_HEADER($1, $2,
guile_cv_func_$1_declared=yes,
_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [patch] 1.6.4: aclocal 1.8 compatibility
2004-07-30 12:16 [patch] 1.6.4: aclocal 1.8 compatibility Maciej W. Rozycki
@ 2004-07-31 1:57 ` Kevin Ryde
2004-08-02 11:21 ` Maciej W. Rozycki
2004-08-12 0:36 ` Rob Browning
0 siblings, 2 replies; 4+ messages in thread
From: Kevin Ryde @ 2004-07-31 1:57 UTC (permalink / raw)
Cc: bug-guile
"Maciej W. Rozycki" <macro@linux-mips.org> writes:
>
> Since automake 1.8 aclocal requires macro names being defined by AC_DEFUN
> to be quoted. Here is a patch that adds missing quotations.
Thanks, I made that change.
(Not sure what automake version will actually be used for the next
release though.)
_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [patch] 1.6.4: aclocal 1.8 compatibility
2004-07-31 1:57 ` Kevin Ryde
@ 2004-08-02 11:21 ` Maciej W. Rozycki
2004-08-12 0:36 ` Rob Browning
1 sibling, 0 replies; 4+ messages in thread
From: Maciej W. Rozycki @ 2004-08-02 11:21 UTC (permalink / raw)
Cc: bug-guile
On Sat, 31 Jul 2004, Kevin Ryde wrote:
> > Since automake 1.8 aclocal requires macro names being defined by AC_DEFUN
> > to be quoted. Here is a patch that adds missing quotations.
>
> Thanks, I made that change.
>
> (Not sure what automake version will actually be used for the next
> release though.)
Well, quoting in this context has always been accepted and perhaps even
preferred, it's only tools that used to be more forgiving, so the patch
doesn't rely on any particular version of automake. But 1.9 has been
released, so you may have a look whether it suits you; otherwise 1.8.5 is
probably a reasonable fall-back.
Newer versions of autotools tend to work better with less common or newer
platforms, e.g. the "mips64el-linux" alias has only recently started being
recognized correctly. It usually requires a complete replacement of
helper files with `libtoolize --automake --copy --force; automake
--add-missing --copy --force', though.
Maciej
_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [patch] 1.6.4: aclocal 1.8 compatibility
2004-07-31 1:57 ` Kevin Ryde
2004-08-02 11:21 ` Maciej W. Rozycki
@ 2004-08-12 0:36 ` Rob Browning
1 sibling, 0 replies; 4+ messages in thread
From: Rob Browning @ 2004-08-12 0:36 UTC (permalink / raw)
Cc: bug-guile, Maciej W. Rozycki
Kevin Ryde <user42@zip.com.au> writes:
> (Not sure what automake version will actually be used for the next
> release though.)
NB: unless there's some compelling reason not to, I generally use
whatever's in current Debian unstable.
--
Rob Browning
rlb @defaultvalue.org and @debian.org; previously @cs.utexas.edu
GPG starting 2002-11-03 = 14DD 432F AE39 534D B592 F9A0 25C8 D377 8C7E 73A4
_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2004-08-12 0:36 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-07-30 12:16 [patch] 1.6.4: aclocal 1.8 compatibility Maciej W. Rozycki
2004-07-31 1:57 ` Kevin Ryde
2004-08-02 11:21 ` Maciej W. Rozycki
2004-08-12 0:36 ` Rob Browning
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).