[Top][All Lists]

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

Re: PATCH: Fix mips*-*-linux*

From: Alexandre Oliva
Subject: Re: PATCH: Fix mips*-*-linux*
Date: 07 Feb 2002 05:11:36 -0200
User-agent: Gnus/5.0805 (Gnus v5.8.5) Emacs/20.7

On Oct 23, 2001, Robert Boehne <address@hidden> wrote:

> Here is the patch reworked for HEAD.  If noone objects
> I'll commit it.

*wakes up*
*looks around*


-- Hmm...  It's been a long time since I last posted in
   libtool-patches.  In fact, it's been a while since I last opened
   its mail folder, so I had completely missed this one.  I wonder if
   it's too late...



*looks at the CVS repo*

-- Hmm...  No, it *was* too late :-(

Oh, well.  I really think this shouldn't have gone in, for the reasons
brought up in gcc-patches and libtool mailing lists.  But, after such
a long silence, I don't feel I'm entitled to just come back and start
checking in or reverting patches, so...  Ok to revert that patch, by
checking this one in?  Tested on athlon-pc-linux-gnu, no regressions,
so this must be right for at least GNU/Linux/x86, right? ;-)

For the record, the reason is that, contrary to the claim in the
patch, on a number of OSs it's not safe to link a static library into
a shared library and, on those in which it is safe, using file_magic
is pointless: pass_all is supposed to behave exactly the way
H.J. intended.

Sorry for having dropped the ball for so long.  It's too bad I don't
foresee this changing in the near future :-(

Index: ChangeLog
from  Alexandre Oliva  <address@hidden>

        Reverted incorrect patch:
        2001-10-24  H.J. Lu  <address@hidden>
        * Allow link against an archive when building a
        shared library.
        * libtool.m4 (lt_cv_deplibs_check_method): Always use
        file_magic for Linux ELF.

RCS file: /cvsroot/libtool/libtool/,v
retrieving revision 1.286
diff -u -p -r1.286
--- 29 Jan 2002 22:58:35 -0000 1.286
+++ 7 Feb 2002 06:58:05 -0000
@@ -2990,13 +2990,6 @@ EOF
                        *) potlib=`$echo "X$potlib" | $Xsed -e 
-                     # It is ok to link against an archive when
-                     # building a shared library.
-                     if $AR -t $potlib > /dev/null 2>&1; then
-                       newdeplibs="$newdeplibs $a_deplib"
-                       a_deplib=""
-                       break 2
-                     fi
                      if eval $file_magic_cmd \"\$potlib\" 2>/dev/null \
                         | ${SED} 10q \
                         | egrep "$file_magic_regex" > /dev/null; then
Index: libtool.m4
RCS file: /cvsroot/libtool/libtool/libtool.m4,v
retrieving revision 1.247
diff -u -p -r1.247 libtool.m4
--- libtool.m4 30 Jan 2002 16:39:24 -0000 1.247
+++ libtool.m4 7 Feb 2002 06:58:08 -0000
@@ -1949,7 +1949,6 @@ linux*)
     # glibc up to 2.1.1 does not perform some relocations on ARM
     lt_cv_deplibs_check_method='file_magic ELF [[0-9]][[0-9]]*-bit [[LM]]SB 
(shared object|dynamic lib )' ;;
-  lt_cv_deplibs_check_method='file_magic ELF [[0-9]][[0-9]]*-bit [[LM]]SB 
(shared object|dynamic lib )'
   lt_cv_file_magic_test_file=`echo /lib/* /lib/libc-*.so`
Alexandre Oliva   Enjoy Guarana', see
Red Hat GCC Developer                  address@hidden,}
CS PhD student at IC-Unicamp        address@hidden,}
Free Software Evangelist                Professional serial bug killer

reply via email to

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