Skip site navigation (1)Skip section navigation (2)
Date:      03 Feb 2002 01:56:33 +0200
From:      Maxim Sobolev <sobomax@mail.ru>
To:        Joe Clarke <marcus@marcuscom.com>
Cc:        gnome@FreeBSD.ORG
Subject:   Re: Missing continue in patch-ltmain.sh
Message-ID:  <1012694178.927.33.camel@notebook>
In-Reply-To: <1012689198.65931.43.camel@shumai.marcuscom.com>
References:  <1012689198.65931.43.camel@shumai.marcuscom.com>

next in thread | previous in thread | raw e-mail | index | archive | help

--=-AzS3U5IkQC9mp7y+bCcl
Content-Type: multipart/mixed; boundary="=-4aVDPLoj3lw4KDuoyHHY"


--=-4aVDPLoj3lw4KDuoyHHY
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

On Sun, 2002-02-03 at 00:33, Joe Clarke wrote:
> Maxim, the recent changes you made to patch-ltmain.sh in a few ports
> have had a bad effect on -stable.  These ports will refuse to link due
> to -pthread missing from the linker arguments.  The problem is the
> continue you added after compiler_flags=3D"$compiler_flags -pthread",
> compiler_flags gets overwritten.  I think the continue should be left
> out when setting compiler_flags.
>=20
> The problem affects glade and mrproject at least.  It looks like any
> port that contains an application will run into this problem in the
> linking phase.  Therefore, gnomevfs is not affected.

Grrr, it seems that libtool doesn't use compiler_flags when linking
executable. Attached please find updated version of patch-ltmain.sh
which should fix the issue. Unfortunately currently I do not have a 4.x
machine to test this patch on, so that it would be really nice if you
could test it with failing ports (mrproject and glade) and let me know
ASAP.

Thanks!

-Maxim
P.S. As the patch-ltmain.sh growth larger and larger I'm more and more
tempted to take several days and rewrite libtool in C specifically for
FreeBSD.


--=-4aVDPLoj3lw4KDuoyHHY
Content-Disposition: attachment; filename=patch-ltmain.sh
Content-Type: text/x-sh; charset=KOI8-R
Content-Transfer-Encoding: quoted-printable


$FreeBSD$

--- ltmain.sh.orig	Wed Jan 23 11:38:40 2002
+++ ltmain.sh	Sun Feb  3 01:44:17 2002
@@ -1037,8 +1037,16 @@
 	continue
 	;;
=20
+      -pthread)
+	compile_command=3D"$compile_command -pthread"
+	finalize_command=3D"$finalize_command -pthread"
+	compiler_flags=3D"$compiler_flags -pthread"
+	continue
+	;;
+
       -module)
 	module=3Dyes
+	build_old_libs=3Dno
 	continue
 	;;
=20
@@ -2412,6 +2420,9 @@
 	  *-*-netbsd*)
 	    # Don't link with libc until the a.out ld.so is fixed.
 	    ;;
+	  *-*-freebsd*)
+	    # FreeBSD doesn't need this...
+	    ;;
 	  *)
 	    # Add libc to deplibs on all other systems if necessary.
 	    if test "$build_libtool_need_lc" =3D "yes"; then
@@ -4179,10 +4190,12 @@
 	fi
=20
 	# Install the pseudo-library for information purposes.
+	if /usr/bin/false; then
 	name=3D`$echo "X$file" | $Xsed -e 's%^.*/%%'`
 	instname=3D"$dir/$name"i
 	$show "$install_prog $instname $destdir/$name"
 	$run eval "$install_prog $instname $destdir/$name" || exit $?
+	fi
=20
 	# Maybe install the static library, too.
 	test -n "$old_library" && staticlibs=3D"$staticlibs $dir/$old_library"

--=-4aVDPLoj3lw4KDuoyHHY--

--=-AzS3U5IkQC9mp7y+bCcl
Content-Type: application/pgp-signature; name=signature.asc
Content-Description: This is a digitally signed message part

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (FreeBSD)
Comment: For info see http://www.gnupg.org

iD8DBQA8XHyfoNu5t4iCBa8RAtMyAJ9GQ/RNhqONDeI3MgjTjYU8lvQymACcCtD3
vOS832G/06sez/UyiIslGVY=
=TL9I
-----END PGP SIGNATURE-----

--=-AzS3U5IkQC9mp7y+bCcl--



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-gnome" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1012694178.927.33.camel>