Tim Van Holder wrote:

On Thu, 2002-10-31 at 05:38, Charles Wilson wrote:

Charles Wilson wrote:


@@ -2243,6 +2254,14 @@
		add="$dir/$linklib"
	      elif test "$hardcode_minus_L" = yes; then
		add_dir="-L$dir"
+		# Try looking first in the location we're being installed to.
+		if test -n "$inst_prefix_dir"; then
+		  case "$libdir" in
+		    [\/]*)

This should use [\\/].  IIRC some versions of bash have a bug in their
matching that causes them not to match a backslash if it is not doubled.

okay, but ...


Also, since this is probably an 'is_absolute' check, it should really be
[\\/]* | ?:[\\/]* )  (cfr the File System Conventions chapter in
the autoconf manual's portability section).

This part won't work. It's possible we need a separate case for A:style paths. Because the rest of the patch does:

+ add_dir="-L$inst_prefix_dir$libdir $add_dir"

E.g. prepend the inst_prefix. But A:/inst_prefix/A:/usr/lib is NOT a valid path; for A:style paths you'd need to strip the drive specifier from $libdir before prepending $inst_prefix...

Help solicited...

--Chuck






_______________________________________________
Libtool mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/libtool

Reply via email to