Re: pkg/45433 (security/cyrus-sasl does not compile on Solaris with lang/gcc34)

[Hans Rosenfeld <hans%NetBSD.org@localhost>]

The following reply was made to PR pkg/45433; it has been noted by GNATS.

From: Hans Rosenfeld <hans%NetBSD.org@localhost>
To: Tim Zingelman <tez%NetBSD.org@localhost>
Cc: gnats-bugs%NetBSD.org@localhost,
        =?iso-8859-1?Q?J=F6rn?= Clausen 
<joern.clausen%uni-bielefeld.de@localhost>
Subject: Re: pkg/45433 (security/cyrus-sasl does not compile on Solaris with 
lang/gcc34)
Date: Sat, 8 Oct 2011 01:48:25 +0200

 On Fri, Oct 07, 2011 at 05:46:31PM -0500, Tim Zingelman wrote:
 > Please get a new libtool and try again to see if this fixes the
 > problem... from the libtool cvs history:
 >
 > 2011-09-16 17:08:27 by Joerg Sonnenberger | Files touched by this commit (2)
 > Log message:
 > Revert. Doesn't follow the patch rules for libtool, wasn't discussed
 > with the OWNER.
 > 
 > 2011-09-14 18:35:51 by Hans Rosenfeld | Files touched by this commit (2)
 > Log message:
 > Don't add -z text unconditionally to ld on SunOS. This make
 > -mimpure-text work again, which is needed for several packages that use
 > assembly optimizations.
 > 
 > (next prior change is May 2011)
 > 
 > I'm also cc'ing Hans, so he is aware the change may have broken more
 > things than the patch rules for libtool.
 
 It is unlikely that the libtool change was related to that, and if it
 was, it would be the other way round. The libtool change was supposed to
 allow working around similar bugs that cannot be easily fixed in the
 source.
 
 Thats what -mimpure-text does, it prevents gcc from adding -z text to
 the linker arguments. Of course that is pretty much useless if libtool
 adds -z text, too, Which is useless in itself as it is the default
 already.
 
 But I think doing that here could be wrong. I have only taken a quick
 look at the bug, and I would suspect a ndbm to be at fault. It's just a
 wild guess, but maybe it's not built as PIC?
 
 
 Hans
 
 
 -- 
 %SYSTEM-F-ANARCHISM, The operating system has been overthrown