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

Reply via email to