On 08/01/07, Robert Story <[EMAIL PROTECTED]> wrote: > > agent -> mibs > > - make_tempfile mibgroup/util_funcs.c > > mibgroup/utilities/execute.c:run_shell_command() > > -Move to agent lib? Duplicate? > > I agree that moving it to the agent library should work. I'd even be ok with > moving it to the core net-snmp library.
Agreed. There's nothing inherently agent-like in creating a temporary file. This could potentially be useful for an application too. +1 for moving it to the core library. > > - init_{module} mibgroup/* + mibgroup/mib_modules_inits.h > > - shutdown_{module} mibgroup/* + mibgroup/mib_modules_shutdown.h Thomas has pointed out that these aren't actually a problem. They're not invoked from a library anyway - the mib_modules.c file is compiled directly into the agent binary itself. > > netsnmp_init_helpers helpers/all_helpers.c snmp_vars.c:init_agent() > > I think the callback fix will work here too, but also wouldn't be terribly > opposed to merging libagent and libhelpers. Agreed. The distinction between libagent and libhelpers doesn't feel as useful as that between these two and libmibs Dave ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Net-snmp-coders mailing list Net-snmp-coders@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/net-snmp-coders