Re: Patch for ext/snmp to support net-snmp-5.0.x
| From: | Harrie Hazewinkel | Date: | Mon, 14 Oct 2002 10:57:57 +0000 |
| Subject: | Re: Patch for ext/snmp to support net-snmp-5.0.x | ||
| References: | 1 2 3 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-89437@lists.php.net to get a copy of this message | ||
HI,
--On Monday, October 14, 2002 10:17 AM +0300 Jani Taskinen <sniper@iki.fi> wrote:
On Mon, 14 Oct 2002, Harrie Hazewinkel wrote:Sorry, if you are offended by me not trying.--On Monday, October 14, 2002 6:10 AM +0300 Jani Taskinen <sniper@iki.fi> wrote:I have not tried this yet, but I taht you do for some include filesFixes the configure/compile problems for me..but I'm a bitObviously not.
But still, for the new net-snmp version is is better to have them all in with the directory prefix. Other package don't do this sometimes and there are sometimes include files with equal names.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..
OK, if you want apply your patch to the core of PHP. I don't see a reason why I need to try your patch. IMHO, it did not solve the include files properly (as earlier indicated). Neither did it solve the version fino string properly.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.
Oops, sorry it is http://www.lisanza.net/php-snmp/php-netsnmp.patch.txtThis patch is available at http://www.lisanza.net/php-snmp/php-snmp.patch.txtThat file does not exist.
I see where this thinking comes from, but Rasmus does not mind I beleive. ;-)) Harrie Internet Management ConsultingThe 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. :)
mailto: harrie@lisanza.net http://www.lisanza.net/-------------------------------------------------------------------- Author of MOD-SNMP, enabling SNMP management the Apache HTTP server