[Buildroot] [PATCH 1/1] sudo: Add ldap support for sudoers rules

[Thomas Petazzoni]

Hello,

On Sat, 28 Jan 2017 15:55:22 +0100, Romain Naour wrote:

> Maybe add a comment here
> 
> comment "ldap integration needs openldap"
> 	depends on !BR2_PACKAGE_OPENLDAP

Well, I'm not against Config.in comments, but in that case, isn't it
sufficiently obvious to the user? Surely if you think about using LDAP
support in sudo, you know that you need to have some kind of LDAP
implementation.

Which makes me think that perhaps a new sub-option is maybe not needed.
Why not just enable LDAP support in sudo when BR2_PACKAGE_OPENLDAP=y ?

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com