From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Sun, 29 Jan 2017 14:18:04 +0100 Subject: [Buildroot] [PATCH 1/1] sudo: Add ldap support for sudoers rules In-Reply-To: References: <1477323263-6689-1-git-send-email-cdf123@cdf123.net> Message-ID: <20170129141804.4749c24d@free-electrons.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net 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