Hi!

It is common practice to write:
--snip--
AC_ARG_ENABLE(.....)dnl
bla bla bla
--snip--
While this works fine with autoconf 2.13, it fails with 2.49a - the configure script
contains syntax errors, ie the shell doesn't find the final "fi" anymore because
it doesn't consider "fiecho" or "fielse" to be a "fi" command.

While the "dnl" may be bad practice, this breaks v 2.13 scripts. Here's a
patch to fix it - just add a semicolon, so no line is wasted, yet no
commands are glued together.

This is basically a resend of
<[EMAIL PROTECTED]>.

Index: ChangeLog
from R�diger Kuhlmann <[EMAIL PROTECTED]>
        * acgeneral.m4(AC_ARG_ENABLE,AC_ARG_WIDTH): Add trailing semicolon to final fi.

Index: acgeneral.m4
===================================================================
RCS file: /cvs/autoconf/acgeneral.m4,v
retrieving revision 1.545
diff -u -u -r1.545 acgeneral.m4
--- acgeneral.m4        2000/08/07 12:33:18     1.545
+++ acgeneral.m4        2000/09/08 18:48:26
@@ -2048,7 +2048,7 @@
   $3
 m4_ifvanl([$4], [else
   $4])dnl
-fi[]dnl
+fi; dnl
 ])# AC_ARG_ENABLE
 
 
@@ -2076,7 +2076,7 @@
   $3
 m4_ifvanl([$4], [else
   $4])dnl
-fi[]dnl
+fi; dnl
 ])# AC_ARG_WITH
 
 AU_DEFUN([AC_WITH],

Yours, R�diger.

-- 
   http://www.ruediger-kuhlmann.de/

Reply via email to