From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Mon, 6 Feb 2017 22:38:06 +0100 Subject: [Buildroot] [PATCH 1/1] sudo: Add ldap support for sudoers rules In-Reply-To: <87r33buhqk.fsf@dell.be.48ers.dk> References: <1477323263-6689-1-git-send-email-cdf123@cdf123.net> <20170206195401.11ccbee3@free-electrons.com> <87r33buhqk.fsf@dell.be.48ers.dk> Message-ID: <20170206223806.5632b9ad@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 Mon, 06 Feb 2017 22:33:39 +0100, Peter Korsgaard wrote: > > I've applied, after removing the explicit Config.in option and instead > > relying on whether BR2_PACKAGE_OPENLDAP is enabled or not to decide if > > ldap support should be enabled or not in sudo. > > Notice that this was discussed during the first review: > > http://lists.busybox.net/pipermail/buildroot/2016-October/175480.html > > Apparently sudo with pam support really needs a configured > /etc/ldap.conf, which might be confusing to people that (perhaps > unrelated to eachother) enabled both sudo and ldap. So in other words we really want to have a sub-option for this situation? Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com