On 09/05/07, Thomas Anders wrote:
> > Log Message:
> > -----------
> > CHANGES: snmpd: PATCH: 1702366: Support for icmpStatsTable
> > Patch provided by Neil Horman



> I think this patch broke the build for non-Linux systems....
> because the "struct icmp_stats_table_entry" definition is inside "#ifdef
> linux". Should we put *more* code inside "#ifdef linux" or is it worth
> to try to extract the portable components and start with *removing* the
> #ifdef around the struct definition?


The quick fix is probably to wrap the handler routine within #ifdef linux
(since that's the only archtiecture that registers it)

If you want to look at implementing the load function for other architectures
too, then feel free - that's clearly the better long-term approach.
But unless you're in a position to do this straight away, I'm inclined to
limit the effects of this patch to just Linux kit for now.

Dave

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/...et-snmp-coders