Re: sed 3.02 required?

[Andreas Jaeger <aj at suse dot de>]

>>>>> Ulrich Drepper writes:

Uli> Andreas Jaeger <aj@suse.de> writes:
>> Which versions are broken?

Uli> Every 1.x and 2.x version.  3.0 and 3.01 were totally broken.

>> And what's the problem?

Uli> You get wrong test cases for gettext.

Can we run those tests only if we found a working sed version?

>> Should we add the version of sed to glibcbug - and document the
>> requirement in the INSTALL document?

Uli> Yeah, might be good.

What do you think of the appended patch?

Btw. I'd like to allow sed 3.02 or newer - otherwise we get again tons
of bugreports because we complain about sed 3.09.

Andreas

2000-02-25  Andreas Jaeger  <aj@suse.de>

	* configure.in: Allow sed 3.02 or newer 3.xx version.

	* manual/install.texi (Tools for Compilation): Document sed
	requirement.

============================================================
Index: manual/install.texi
--- manual/install.texi	2000/01/12 10:19:48	1.37
+++ manual/install.texi	2000/02/25 15:04:34
@@ -392,6 +392,16 @@
 Perl is not required, but it is used if present to test the
 installation.  We may decide to use it elsewhere in the future.
 
+@item
+GNU @code{sed} 3.02 or newer
+
+Sed is used in several places to generate files.  Most scripts work with
+any version of @code{sed}.  The known exception is the script
+@code{po2test.sed} in the @code{intl} subdirectory which is used to
+generate @code{msgs.h} for the testsuite.  This script works correctly
+only with GNU @code{sed} 3.02.  If you like to run the testsuite, you
+should definitly upgrade @code{sed}.
+
 @end itemize
 
 @noindent
============================================================
Index: configure.in
--- configure.in	2000/02/11 18:38:31	1.269
+++ configure.in	2000/02/25 15:04:37
@@ -514,7 +514,7 @@
   [3.1[1-9] | 3.[2-9][0-9] | 4.* | 1.6[89] | 1.7[0-9]], MAKEINFO=: aux_missing=t)
 AC_CHECK_PROG_VER(SED, gsed sed, --version,
   [GNU sed version \([0-9]*\.[0-9.]*\)],
-  [3.02], SED=: aux_missing=t)
+  [3.0[2-9]|3.[1-9]*], SED=: aux_missing=t)
 
 if test -n "$critic_missing"; then
 AC_MSG_ERROR([

-- 
 Andreas Jaeger
  SuSE Labs aj@suse.de
   private aj@arthur.rhein-neckar.de