Re: Patch for ext/snmp to support net-snmp-5.0.x

On Mon, 14 Oct 2002, Harrie Hazewinkel wrote: >HI > >--On Monday, October 14, 2002 6:10 AM +0300 Jani Taskinen <sniper@iki.fi> >wrote: > >> >> This patch: >> >> >> http://www.kolumbus.fi/jani.taskinen/bugs/php_snmp.patch.gz >> >> Fixes the configure/compile problems for me..but I'm a bit > >I have not tried this yet, but I taht you do for some include files Obviously not. >the directory prefix net-snmp where others >don't have it. The idea behind the include files in net-snmp >is that all go into a net-snmpdirectory and so can be distiguished >better from others. Huh? Maybe you missed the #ifdef HAVE_NET_SNMP part there? Those without the prefix are for ucd-snmp.. >I have also made some patch available a time ago in which >from the configure you could select either the old ucd-snmp >or the new net-snmp libraries. As you propably didn't use the obvious net-snmp-config script which is good enough indication that we're dealing with net-snmp. Thus there is no need for such hacky options. Please try the patch. >This patch is available at >http://www.lisanza.net/php-snmp/php-snmp.patch.txt That file does not exist. >The only thing is I would like to know from this group whether >it would be OK to patch up the code to use only NET-SNMP now or still >both. I'd say dump the ucd-snmp support, but I bet Rasmus won't like that idea. :) --Jani