svn commit: r250379 - in head/usr.sbin/bsnmpd/modules: . snmp_hast

[Konstantin Belousov]

On Thu, May 09, 2013 at 10:14:14AM +0300, Mikolaj Golub wrote:
> On Wed, May 08, 2013 at 04:44:54PM -0700, Peter Wemm wrote:
> > On Wed, May 8, 2013 at 4:25 PM, Peter Wemm <peter at wemm.org> wrote:
> > > On Wed, May 8, 2013 at 1:03 PM, Mikolaj Golub <trociny at freebsd.org> wrote:
> > >> Author: trociny
> > >> Date: Wed May  8 20:03:37 2013
> > >> New Revision: 250379
> > >> URL: http://svnweb.freebsd.org/changeset/base/250379
> > >>
> > >> Log:
> > >>   HAST module for bsnmpd(1).
> > >>
> > >>   Reviewed by:  harti, pjd
> > >>   MFC after:    2 weeks
> > >
> > > This breaks world on just about everything other than i386, and it is
> > > technically broken there too but doesn't actually cause a build
> > > failure.
> > >
> > > You cannot link a .so file against a non-pic library, libl.a in this case.
> > >
> > > ===> usr.sbin/bsnmpd/modules/snmp_hast (all)^M
> > > /usr/obj/usr/src/tmp/usr/bin/ld: /usr/obj/usr/src/tmp/usr/lib/libl.a(libyywrap.o
> > > ): relocation R_X86_64_32 against `a local symbol' can not be used when making a
> > >  shared object; recompile with -fPIC^M
> > > /usr/obj/usr/src/tmp/usr/lib/libl.a: could not read symbols: Bad value^M
> > > *** [snmp_hast.so.6] Error code 1^M
> > 
> > 
> > Hmm. It seems clang optimizes something away that gcc doesn't.  When
> > building with clang the problem doesn't show up.
> 
> Sorry for breaking build. I am always running tinderbox before commits
> like this, which did not help this time. Now I will run it twice, with
> both clang and gcc.
> 
> Thank you for r250394. I will investigate how to fix this. Any
> suggestions are highly appreciated.

Why do you need libl.a ? If only for yywrap(), then re-implementing
it for the module seems to be trivial.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-all/attachments/20130509/4ef1c5ac/attachment.sig>