Friday, July 6, 2018 8:17 AM

> These first two patches are fine, and I'll merge them soon.

Great

> The code in snmpd.c looks fine to me.

It´s not that complicated when using this AgentX master agent

> The name 'snmpd' is a not a good choice.  The net-snmp project already
> delivers a program of that name.  We are building an AgentX, are we
> not?  So let's use a unique name that is also more descriptive.

That's correct. No problem changing the name

> How about 'ptpax' for PTP AgentX?

I have no problem with the name, I can update the patches for it

> I've also looked at the build issues, and I have an idea how to
> simplify the test for including snmp support and also support cross.
> I'll post details as soon as we can agree on a name for the agentx
> program.

Great, Lets go for ptpax

Can you merge the first two patches and I will make a new serie from patch
3 -  snmpd: Add snmp sub agent

/Anders


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to