Re: [ptxdist] [PATCH] lldpd: add package

[Alexander Dahl <post@lespocky.de>]

Hei Michael, 

Am 2013-09-09 16:45, schrieb Michael Olbrich:
> please change the subject to "lldpd: ..."

I'll do it in the next patch, I noticed this right after sending it. :-/

>> Yes. This is tricky and didn't have a good idea how to check it. There's
>> the parameter NET_SNMP_MIB_MODULES which is used in this parameter in
>> the prepare stage:
>>
>>   --with-mib-modules=$(PTXCONF_NET_SNMP_MIB_MODULES)
>>
>> You have to add/include "agentx" in the ptxdist menuconfig entry
>> NET_SNMP_MIB_MODULES. You can add other and there's no simple switch for
>> each of them. This is the way to enable several things in net-snmp and
>> lldpd seems to check it the right way, but I had no idea how to
>> "translate" this mechanism to the ptxdist layer above.
> 
> Make an option for "agentx", so you can select it.

This would be an option in net-snmp package and this is the difficulty.
At the moment there's PTXCONF_NET_SNMP_MIB_MODULES which is a space
separated list. From the `./configure --help` of net-snmp:

  --with-mib-modules="item1 ..."  Compile with additional mib modules
                                    (Space separated list).
  --with-out-mib-modules="list"   Compile without these mib modules.

Possible entries make a list of two screen pages, I guess around 30
entries (and "agentx" is one of them). If I choose one of them to be a
separate option in ptxdist net-snmp conf (maybe as submenu) while
keeping the list in PTXCONF_NET_SNMP_MIB_MODULES, we would have to test
the new option in net-snmp.make and add it to the list if selected. But
what if someone puts the same option in the list?

Or would it be better to put variables for almost 30 options to the
ptxdist config?

However this is no change in lldpd package but in net-snmp, which would
have to be made first.

>> This is the default lldpd uses. See the output of ./configure from
>> lldpd:
>>
>>   --with-privsep-user     Which user to use for privilege separation
>>                           [default=_lldpd]
>>   --with-privsep-group    Which group to use for privilege separation
>>                           [default=_lldpd]
>>   --with-privsep-chroot   Which directory to use to chroot lldpd
>>                           [default=/var/run/lldpd]
>>
>> Actually I set this user to nobody and the group to daemon. As far as I
>> understood lldpd this is hardcoded and I wanted to have it configurable
>> so I added it to the ptxdist menu and copied the defaults from lldpd.
>> Maybe the do not make sense for most of the users. Do you have a better
>> suggestion for this user and group?
> 
> use "nobody" by default. Otherwise the defaults break targetinstall

Okay, and for the group?

>> > no "default n" needed. Thats the default anyways.
>>
>> Good to know, but for me the code of the in rule is more clear if I add
>> this. However, if you want to have it removed, I'll remove it.
> 
> thanks.

Okay I'll remove every "default n" in the next patch.

Greets
Alex

-- 
»With the first link, the chain is forged. The first speech censured,
the first thought forbidden, the first freedom denied, chains us all
irrevocably.« (Jean-Luc Picard, quoting Judge Aaron Satie)
*** GnuPG-FP: 02C8 A590 7FE5 CA5F 3601  D1D5 8FBA 7744 CC87 10D0 ***

-- 
ptxdist mailing list
ptxdist@pengutronix.de