Wes Hardaker wrote: > I'd rather still try to go down this line as it seems to be the > cleanest, IMHO. I'd propose that in the system .h files we should > require defining the os definitions in the same way... IE, > > #define osf4 > > is in there now but I think it should be: > > #define osf4 osf4 > > This seems much safer in general and I'm actually amazed we haven't > been bitten by this bug yet anyway (you'd think function names, etc, > would have hit this at some point already)
I agree. Any objections to change include/net-snmp/system/*.h accordingly? Then, still, shouldn't we also change MODULECPP="$CPP -D$PARTIALTARGETOS ..." to MODULECPP="$CPP -D$PARTIALTARGETOS=$PARTIALTARGETOS ..." in configure[.in] (and whereever else we follow a similar approach)? +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=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Net-snmp-coders mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
