bug-libtool
[Top][All Lists]
Advanced

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

Re: libtool assumes 'egrep' in a couple of places


From: Scott James Remnant
Subject: Re: libtool assumes 'egrep' in a couple of places
Date: Tue, 03 Feb 2004 19:55:22 +0000

On Fri, 2004-01-30 at 22:50, Paul Eggert wrote:

> This is just a mostly-theoretical porting bug.  POSIX 1003.1-2001 has
> withdrawn "egrep"; it is no longer required on a standard POSIX
> system.  Here's a patch to libtool so that it no longer assumes the
> existence of egrep.
> 
This only affects Libtool 1.5.x -- HEAD already has the offending cases
removed.  I also found an occurrence of 'grep -E' in ltmain.in which
I've replaced with $EGREP.

I've used the following slightly more verbose ChangeLog entry and
applied to branch-1-5.

2004-2-03  Paul Eggert  <address@hidden>

        * libtool.m4 (_LT_AC_LANG_CXX_CONFIG, AC_LIBTOOL_PROG_LD_SHLIBS):
        Replace needless uses of egrep with grep as egrep may not exist,
        POSIX no longer mandates it.
        * ltmain.in: Replace use of 'grep -E' with '$EGREP -e' in case
        egrep is the only thing that exists.

----8<--------8<--------8<--------8<--------8<--------8<--------8<--------8<----
diff -u -r1.334.2.19 ltmain.in
--- ltmain.in   25 Jan 2004 00:26:06 -0000      1.334.2.19
+++ ltmain.in   3 Feb 2004 19:49:22 -0000
@@ -131,7 +131,7 @@
     ;;
   *ar\ archive*) # could be an import, or static
     if eval $OBJDUMP -f $1 | $SED -e '10q' 2>/dev/null | \
-      grep -E 'file format pe-i386(.*architecture: i386)?' >/dev/null ; then
+      $EGREP -e 'file format pe-i386(.*architecture: i386)?' >/dev/null ; then
       win32_nmres=`eval $NM -f posix -A $1 | \
        sed -n -e '1,100{/ I /{x;/import/!{s/^/import/;h;p;};x;};}'`
       if test "X$win32_nmres" = "Ximport" ; then
---->8-------->8-------->8-------->8-------->8-------->8-------->8-------->8----

Scott
-- 
Have you ever, ever felt like this?
Had strange things happen?  Are you going round the twist?

Attachment: signature.asc
Description: This is a digitally signed message part


reply via email to

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