On Wed, Oct 13, 2010 at 12:16 PM, Wes Hardaker
<harda...@users.sourceforge.net> wrote:
>
> BF> I updated net-snmp to 5.6, and all of my net-snmp consumers broke.  It
> BF> looks like <net-snmp/library/snmp_api.h> (included from
> BF> <net-snmp/session_api.h> accidentally used "netsnmp_transport *"
> BF> instead of "struct netsnmp_transport_s *":
>
> Hi Bill!
>
> Though I think your patch is fine, I find it odd that you're the only
> person that has hit this particular problem.

Yes, I was surprised too.  I'm guessing that others who use the
net-snmp APIs copy more code from, e.g., the utility programs, while
we wrote at least some of our stuff from scratch using the API
documentation.

>  I suspect if you had
> included "net-snmp-includes.h" instead of just the session_api.h it
> wouldn't have been a problem (not that you should be required to do
> that).

In this particular chunk of code, we use

#include <net-snmp/net-snmp-config.h>
#include <net-snmp/library/snmp_alarm.h>
#include <net-snmp/session_api.h>

(I have no principled reason for why it is that way.)

> Also, I bet if you had included net-snmp/types.h first it would have
> worked as well.
>
> Anyway, I'm applying your patch for the 5.6.1 release.

Thanks!

  Bill

------------------------------------------------------------------------------
Download new Adobe(R) Flash(R) Builder(TM) 4
The new Adobe(R) Flex(R) 4 and Flash(R) Builder(TM) 4 (formerly 
Flex(R) Builder(TM)) enable the development of rich applications that run
across multiple browsers and platforms. Download your free trials today!
http://p.sf.net/sfu/adobe-dev2dev
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to