Re: rfv: fix more lib dependencies

Wes Hardaker wrote:
> I'm more inclined to vote in favor of it if you could list all the
> platforms that such a feature is tested on, including the oldest ones
> you could find that support shared libraries.
>
> Even if you could convince me it should be left on by default, I'm
> still not entirely sure this should be put in until the next release.
> Changing how we call linkers this late into a release just doesn't
> give me warm fuzzies when I sleep at night. It has nothing to do with
> whether or not I think it should be done, it has to do with whether I
> think it should be done now.

Like I said, adding if/else/endif blocks to make the proposed
libwrap/libadm/libnm/libperfstat linking depend on whether --enable-as-needed
has been given (which defaults to off, like currently done for
libcrypto/libsensors), should be pretty straightforward once we agree that's the
way to go.

May we interpret your vote as a "+1" for going this route, i.e.
- disable the proposed linking changes by default (because you think it may not
work on on some ancient platforms)
- enable the proposed linking changes if --enable-as-needed is given (because
it's the Right Thing and more convenient)
?

We'd be at +3, then.

Unfortunately I'll not have further time this weekend to do the necessary
changes to the patch. Would anyone else be willing to do it (Wes? darix?)? I
could offer to run build tests with the enhanced patch, then, at least.


+Thomas

--
Thomas Anders (thomas.anders at blue-cable.de)

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=...057&dat=121642
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/...et-snmp-coders