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

From: 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:
--On Monday, October 14, 2002 6:10 AM +0300 Jani Taskinen <sniper@iki.fi> wrote:
    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.
Sorry, if you are offended by me not trying.
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..
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.
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.
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.
This patch is available at http://www.lisanza.net/php-snmp/php-snmp.patch.txt
    That file does not exist.
Oops, sorry it is http://www.lisanza.net/php-snmp/php-netsnmp.patch.txt
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. :)
I see where this thinking comes from, but Rasmus does not mind I beleive. ;-)) Harrie Internet Management Consulting
mailto: harrie@lisanza.net                   http://www.lisanza.net/
-------------------------------------------------------------------- Author of MOD-SNMP, enabling SNMP management the Apache HTTP server

« previous php.dev (#89437) next »