On Tue, Aug 22, 2006 at 09:26:03AM +0100, Dave Shield wrote: > > TA> It's a start, but I think our current approach of shipping all > > TA> autoconf variables in net-snmp-config.h is even more flawed than > > TA> that. See some random Debian bug (wrt. some other software) on the > > TA> topic: > > > > TA> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=363173 > > Is it really that unimaginable that a different package might also > define LOGFILE? > Why should someone have to go to all of the work of analysing our > config file, and picking out all of the relevant definitions - just > because we were too lazy to change this to something more unique, like > NETSNMP_LOGFILE?
I am strongly in favour of liberally sprinkle net-snmp-config.h with NETSNMP_ prefixes. > Let's take a step back :-) What is the real problem here? > > As I see it, there are three general types of entry in the config.h file: > > a) Net-SNMP-specific settings - stuff that's only meaningful to our code. > (e.g. DEFAULT_MIBDIRS, USE_REVERSE_ASNENCODING, etc) > > It probably doesn't matter if an application includes these, as > they're unlikely to clash with anything from another autoconf-based > library. > > b) Generic system-related settings > (e.g. HAVE_SOME_HEADER_H, HAVE_SOME_FUNCTION) > > It probably doesn't matter if an application includes there, as they > ought to be consistent with the equivalent settings from another > autoconf-based library. Maybe the compilation would throw up a few > warnings, but nothing too critical. The problem with shipping of the HAVE_s is that they reflect the state on the build machine as opposed to the install machine and in those days of package managers the build and install machine often do differ. This leads to the next problem - a package using net-snmp might get confused since net-snmp-config.h defines a HAVE_ that configure left undefined since that package isn't installed on the machine. I think we should move all the HAVE_s to a separate config file that isn't installed so that only macros named NETSNMP_* are present in net-snmp-config.h /MF ------------------------------------------------------------------------- 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=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Net-snmp-coders mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
