libtool-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: some missing _LT_DECL_SED requirements


From: Ralf Wildenhues
Subject: Re: some missing _LT_DECL_SED requirements
Date: Wed, 17 May 2006 19:33:16 +0200
User-agent: Mutt/1.5.11

* Gary V. Vaughan wrote on Wed, May 17, 2006 at 07:17:30PM CEST:
> Ralf Wildenhues wrote:
> >Eric reported this against CVS M4 recently:
> >http://lists.gnu.org/archive/html/m4-patches/2006-05/msg00005.html
> >
> >OK?
> 
> Yes please.

Except I forgot that the missing bit was $Xsed, not $SED.
So we'd also need:

> >     * libltdl/m4/libtool.m4 (_LT_LINKER_OPTION): Require
> >     _LT_DECL_SED.
> >     (_LT_CMD_GLOBAL_SYMBOLS, _LT_LINKER_SHLIBS): Likewise.
> >     * libltdl/m4/ltdl.m4 (LTDL_INIT): Likewise.
> >     Report by Eric Blake.
        (_LT_DECL_SED): Set $Xsed here, using $SED.  Moved from...
        (_LT_SETUP): ...here.


Index: libltdl/m4/libtool.m4
===================================================================
RCS file: /cvsroot/libtool/libtool/libltdl/m4/libtool.m4,v
retrieving revision 1.67
diff -u -r1.67 libtool.m4
--- libltdl/m4/libtool.m4       17 May 2006 14:28:40 -0000      1.67
+++ libltdl/m4/libtool.m4       17 May 2006 17:32:01 -0000
@@ -181,7 +181,6 @@
 
 # Sed substitution that helps us do robust quoting.  It backslashifies
 # metacharacters that are still active within double-quoted strings.
-Xsed='sed -e 1s/^X//'
 sed_quote_subst='s/\([["`$\\]]\)/\\\1/g'
 
 # Same as above, but do not quote variable references.
@@ -6796,6 +6795,7 @@
 m4_defun([_LT_DECL_SED],
 [AC_PROG_SED
 test -z "$SED" && SED=sed
+Xsed="$SED -e 1s/^X//"
 _LT_DECL([], [SED], [1], [A sed program that does not truncate output])
 _LT_DECL([], [Xsed], ["\$SED -e 1s/^X//"],
     [Sed that helps us avoid accidentally triggering echo(1) options like -n])




reply via email to

[Prev in Thread] Current Thread [Next in Thread]