Re: gettext libintl.la dependency_libs problem

[Albert Chin]

On Thu, Jan 02, 2003 at 10:26:28PM -0600, Tim Mooney wrote:
> In regard to: Re: gettext libintl.la dependency_libs problem, Albert Chin...:
> 
> >On Thu, Jan 02, 2003 at 07:07:42PM -0600, Albert Chin wrote:
> >> On Thu, Jan 02, 2003 at 06:30:58PM -0600, Tim Mooney wrote:
> >> > ...
> >> >
> >> > Now say you build some package (for me it was glib-2.2.0) that depends on
> >> > gettext.  When you go to link the package on a platform like Tru64 or
> >> > IRIX, the link will fail because libtool passes the 
> >> > `-R/usr/local/gnu/lib'
> >> > to whatever is doing the linking on the platform in question (ld
> >> > generally).
> >> >
> >> > Many vendor compilers and linkers don't understand `-R', so the link
> >> > bombs.
> >
> >Show the command-line doing this.
> 
> With versioning this gets pretty long, hopefully it comes through OK:
> 
> ...

According to the libtool info file:
  `-R LIBDIR'
       If OUTPUT-FILE is a program, add LIBDIR to its run-time path.  If
       OUTPUT-FILE is a library, add -RLIBDIR to its DEPENDENCY_LIBS, so
       that, whenever the library is linked into a program, LIBDIR will
       be added to its run-time path.

So, I guess it's a bug in libtool. Is the patch below correct? We can
throw away -R in $deplibs because we already added it to $xrpath when
we scanned $dependency_libs.

I haven't tested the patch mutch.

-- 
albert chin (address@hidden)

-- snip snip
Index: ltmain.in
===================================================================
RCS file: /cvsroot/libtool/libtool/ltmain.in,v
retrieving revision 1.259.2.26
diff -u -3 -p -r1.259.2.26 ltmain.in
--- ltmain.in   11 Oct 2002 16:06:07 -0000      1.259.2.26
+++ ltmain.in   3 Jan 2003 08:21:16 -0000
@@ -2084,6 +2084,7 @@ compiler."
          for deplib in $tmp_libs; do
            case $deplib in
            -L*) new_libs="$deplib $new_libs" ;;
+           -R*) ;;
            *)
              case " $specialdeplibs " in
              *" $deplib "*) new_libs="$deplib $new_libs" ;;