Ethereal-users: Re: [Ethereal-users] Re: updated net-snmp/ethereal patch

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next
From: Guy Harris <gharris@xxxxxxxxx>
Date: Thu, 24 Oct 2002 00:16:31 -0700
On Wed, Oct 23, 2002 at 11:25:35AM -0700, Guy Harris wrote:
> On Wed, Oct 23, 2002 at 07:55:03AM -0700, Wes Hardaker wrote:
> > Ok, how's this for another wack at it.  Untested with ucd-snmp and on
> > windows, but should be ok (then again, I said that last time).
> 
> Well, it worked on Solaris 8 with UCD-SNMP; my Windows development
> environment is at home, so I can't test it yet.

It didn't build on Windows, at least not with the version of Net-SNMP I
have.

I checked in some changes:

	I wrapped the stuff to get the header file to get the SNMP
	version in #ifdef HAVE_SOME_SNMP/#endif, and made it include
	<net-snmp/version.h> if HAVE_NET_SNMP is defined (as, otherwise,
	"netsnmp_get_version()" isn't declared).  (We only support UCD
	SNMP versions that have "sprint_realloc_objid()"; do those
	versions all install <ucd-snmp/version.h>? If so, we don't have
	to check whether that header file is available, we can just
	#include it.)  That was done in tethereal.c, gtk/main.c, and
	gtk2/main.c.

	I included <net-snmp/config_api.h> in "packet-snmp.c" if we're
	using Net-SNMP, to declare "read_premib_configs()" and
	"read_configs()".

	I made gtk/Makefile.nmake and gtk2/Makefile.nmake add /I flags
	for the Net-SNMP include directories.

With those changes, it built on my machine.