- add fix for same line comments #449245 (Ralf Wildenhues)

This commit is contained in:
Karsten Hopp 2008-06-24 13:53:57 +00:00
parent 295e299a27
commit 6f8699b2eb
2 changed files with 81 additions and 1 deletions

View File

@ -0,0 +1,75 @@
diff -urN autoconf-2.62/ChangeLog autoconf-2.62_new/ChangeLog
--- autoconf-2.62/ChangeLog 2008-06-24 15:18:34.000000000 +0200
+++ autoconf-2.62_new/ChangeLog 2008-06-24 15:24:35.000000000 +0200
@@ -1,3 +1,14 @@
+2008-06-16 Ralf Wildenhues <Ralf.Wildenhues@gmx.de>
+
+ Fix '#undef variable /* comment */' transform in config headers.
+ * lib/autoconf/status.m4 (_AC_OUTPUT_HEADERS_PREPARE): For
+ undefined preprocessor macros that are followed by a comment
+ in the header template, do not create nested comments in the
+ output.
+ * tests/torture.at (@%:@define header templates): Extend test.
+ * NEWS: Update.
+ Report by Karsten Hopp <karsten@redhat.com>.
+
2008-06-05 Eric Blake <ebb9@byu.net>
Fix regression in AT_KEYWORDS([Macro]), from 2007-10-18.
diff -urN autoconf-2.62/lib/autoconf/status.m4 autoconf-2.62_new/lib/autoconf/status.m4
--- autoconf-2.62/lib/autoconf/status.m4 2008-04-06 01:04:48.000000000 +0200
+++ autoconf-2.62_new/lib/autoconf/status.m4 2008-06-24 15:21:54.000000000 +0200
@@ -832,9 +832,9 @@
}
split(mac1, mac2, "(") #)
macro = mac2[1]
+ prefix = substr(line, 1, index(line, defundef) - 1)
if (D_is_set[macro]) {
# Preserve the white space surrounding the "#".
- prefix = substr(line, 1, index(line, defundef) - 1)
print prefix "define", macro P[macro] D[macro]
next
} else {
@@ -842,7 +842,7 @@
# in the case of _POSIX_SOURCE, which is predefined and required
# on some systems where configure will not decide to define it.
if (defundef == "undef") {
- print "/*", line, "*/"
+ print "/*", prefix defundef, macro, "*/"
next
}
}
diff -urN autoconf-2.62/NEWS autoconf-2.62_new/NEWS
--- autoconf-2.62/NEWS 2008-06-24 15:18:34.000000000 +0200
+++ autoconf-2.62_new/NEWS 2008-06-24 15:25:28.000000000 +0200
@@ -25,6 +25,9 @@
like '__restrict' if the variant spelling is available, as this is
more likely to work when mixing C and C++ code.
+** Config header templates `#undef UNDEFINED /* comment */' do not lead to
+ nested comments any more; regression introduced in 2.62.
+
** AC_CHECK_ALIGNOF's type argument T is now documented better: it must
be a string of tokens such that "T y;" is a valid member declaration
in a struct.
diff -urN autoconf-2.62/tests/torture.at autoconf-2.62_new/tests/torture.at
--- autoconf-2.62/tests/torture.at 2008-04-06 01:04:48.000000000 +0200
+++ autoconf-2.62_new/tests/torture.at 2008-06-24 15:23:21.000000000 +0200
@@ -497,6 +497,8 @@
#define str(define) \
#define
#define stringify(arg) str(arg)
+#undef aaa /* with comments */
+#undef not_substed /* with comments */
]])
AT_CHECK_AUTOCONF
@@ -527,6 +529,8 @@
#define str(define) \
#define
#define stringify(arg) str(arg)
+#define aaa AAA
+/* #undef not_substed */
]])
AT_CHECK([cat config.h], 0, expout)

View File

@ -1,7 +1,7 @@
Summary: A GNU tool for automatically configuring source code Summary: A GNU tool for automatically configuring source code
Name: autoconf Name: autoconf
Version: 2.62 Version: 2.62
Release: 2%{?dist} Release: 3%{?dist}
License: GPLv2+ and GFDL License: GPLv2+ and GFDL
Group: Development/Tools Group: Development/Tools
Source: http://ftp.gnu.org/gnu/autoconf/autoconf-%{version}.tar.bz2 Source: http://ftp.gnu.org/gnu/autoconf/autoconf-%{version}.tar.bz2
@ -9,6 +9,7 @@ Source1: filter-provides-automake.sh
Source2: filter-requires-automake.sh Source2: filter-requires-automake.sh
# This one will be included in 2.63: # This one will be included in 2.63:
Patch0: autoconf-2.62-449973.patch Patch0: autoconf-2.62-449973.patch
Patch1: autoconf-2.62-449245.patch
URL: http://www.gnu.org/software/autoconf/ URL: http://www.gnu.org/software/autoconf/
BuildRequires: m4 >= 1.4.7, emacs BuildRequires: m4 >= 1.4.7, emacs
Requires: m4 >= 1.4.7, mktemp, imake, grep Requires: m4 >= 1.4.7, mktemp, imake, grep
@ -41,6 +42,7 @@ their use.
%prep %prep
%setup -q %setup -q
%patch0 -p1 %patch0 -p1
%patch1 -p1
%build %build
# use ./configure here to avoid copying config.{sub,guess} with those from the # use ./configure here to avoid copying config.{sub,guess} with those from the
@ -82,6 +84,9 @@ fi
%doc AUTHORS COPYING ChangeLog NEWS README THANKS TODO %doc AUTHORS COPYING ChangeLog NEWS README THANKS TODO
%changelog %changelog
* Tue Jun 24 2008 Karsten Hopp <karsten@redhat.com> 2.62-3
- add fix for same line comments #449245 (Ralf Wildenhues)
* Fri Jun 06 2008 Karsten Hopp <karsten@redhat.com> 2.62-2 * Fri Jun 06 2008 Karsten Hopp <karsten@redhat.com> 2.62-2
- add upstream fix from Eric Blake for #449973, - add upstream fix from Eric Blake for #449973,
m4_if releated error message from autotest m4_if releated error message from autotest