Date: Thu, 2 Jun 2011 13:06:17 +0300 From: Kostik Belousov <kostikbel@gmail.com> To: Jeremie Le Hen <jeremie@le-hen.org> Cc: kan@freebsd.org, freebsd-hackers@freebsd.org Subject: Re: compiling ports with SSP (was: [PATCH] Add -lssp_nonshared to GCC's LIB_SPEC unconditionally)= Message-ID: <20110602100617.GF48734@deviant.kiev.zoral.com.ua> In-Reply-To: <20110601214723.GD53247@felucia.tataz.chchile.org> References: <20100919184146.GE2389@deviant.kiev.zoral.com.ua> <20100920162925.GL6864@felucia.tataz.chchile.org> <20100920192708.GK2389@deviant.kiev.zoral.com.ua> <20100927094651.GB57265@felucia.tataz.chchile.org> <20100927154457.GJ43070@deviant.kiev.zoral.com.ua> <20101005181804.GJ7536@felucia.tataz.chchile.org> <20101105213905.GT30284@felucia.tataz.chchile.org> <20101105190023.29e5d39d@kan.dnsalias.net> <20101106194702.GN2392@deviant.kiev.zoral.com.ua> <20110601214723.GD53247@felucia.tataz.chchile.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--OItkzkEMg3BnaIAB Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jun 01, 2011 at 11:47:23PM +0200, Jeremie Le Hen wrote: > Hi Kostik, >=20 > Thanks to b.f., I've been reminded that this patch has yet to be > committed :). >=20 > As a reminder, here are the archive pointers to the discussion: > http://lists.freebsd.org/pipermail/freebsd-hackers/2010-August/032549.html > continued... > http://lists.freebsd.org/pipermail/freebsd-hackers/2010-September/033028.= html > continued... > http://lists.freebsd.org/pipermail/freebsd-hackers/2010-November/033478.h= tml >=20 > On Sat, Nov 06, 2010 at 09:47:02PM +0200, Kostik Belousov wrote: > > On Fri, Nov 05, 2010 at 07:00:23PM -0400, Alexander Kabaev wrote: > > > On Fri, 5 Nov 2010 22:39:06 +0100 > > > Jeremie Le Hen <jeremie@le-hen.org> wrote: > > >=20 > > > > Hi Kib, > > > >=20 > > > > On Tue, Oct 05, 2010 at 08:18:04PM +0200, Jeremie Le Hen wrote: > > > > >=20 > > > > > On Mon, Sep 27, 2010 at 06:44:57PM +0300, Kostik Belousov wrote: > > > > > > Hardcoding /usr/lib as the path to the library in the script lo= oks > > > > > > problematic. For the buidlworld, you are linking resulting > > > > > > binaries with the host library, instead of the > > > > > > buildworld-produced one. For lib32, it makes non-working > > > > > > combination of 32/64 bit. > > > > >=20 > > > > > Sorry for the late reply, but I had to collect various evidences > > > > > for my sayings and my development machine is reaaaaaaaaaaally slo= w. > > > > >=20 > > > > > In fact it seems the toolchain built for buildworld contains a ld= (1) > > > > > binary which invariably bases lookups for libraries in ${WORLDTMP= }, > > > > > even in case of an absolute path. I have two evidences of this: > > > > > - Putting /usr/obj/usr/src/tmp/usr/lib/libssp_nonshared.a in > > > > > /usr/obj/usr/src/tmp/usr/lib/libc.ld leads toolchain's ld(1) to > > > > > use /usr/obj/usr/src/tmp/usr/obj/usr/src/tmp/usr/lib/libssp_nonsh= ared.a; > > > > > - I also verified this with a hand-wrought opensnoop-like DTrace > > > > > script. > > > >=20 > > > > I dare to remind you about my patch. Do you have any other concern= s? > > > >=20 > > > > Thanks. > > > > Regards, > > > > --=20 > > > > Jeremie Le Hen > > > >=20 > > > > Humans are born free and equal. But some are more equal than other= s. > > > > Coluche > > >=20 > > > Hmm, I thought I did approve this patch already a long time agi, but > > > since you asked: > > >=20 > > > +.if defined(SHLIB_LDSCRIPT) && exists(${.CURDIR}/${SHLIB_LDSCRIPT}) > > >=20 > > > this should be: > > >=20 > > > +.if defined(SHLIB_LDSCRIPT) > > >=20 > > > ditto for all other similar places. Otherwise I do not think we should > > > hold the patch in queue ans should unleash it on unsuspecting public. > >=20 > > Also, I think the "DEBUG" lines should be removed. >=20 > Sure, I'll do it in my next update. >=20 > > You install the libxxx.ld and then symlink libxxx.so to libxxx.ld. > > Why ? Would it be enough to install just the libxxx.so ? >=20 > I just thought it would be less puzzling for users than noticing that > libc.so is only a few hundred of ascii. I don't have a strong opinion > about this though. I prefer to not have a symlink. >=20 > > Otherwise, I think you need the similar > > .if ${SHLIBDIR} =3D=3D ${LIBDIR} > > magic, that is better to be avoided. >=20 > Can you explain a little bit more about this one please? I'm willing to > post an updated patch for further review.=20 I think this comment was somehow related to the fact that make install does not work from the buildenv, or something similar. It was too long time ago for me to remember details. Also, I remember there was a concern about linker script not being installed in the cross-build environment during buildworld, or something close to what I stated. Was it resolved ? [I pinged you some time ago, you did not responded]. --OItkzkEMg3BnaIAB Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (FreeBSD) iEYEARECAAYFAk3nYJkACgkQC3+MBN1Mb4h3WQCbBW5R7cvmvklT21YWMArLmH8/ F+gAoM3HrLgMK7Z62SzSBTiBl1d67QF1 =hdXj -----END PGP SIGNATURE----- --OItkzkEMg3BnaIAB--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110602100617.GF48734>