[Buildroot] [PATCH 2/2] lighttpd: add optional pam support

[Peter Korsgaard]

>>>>> "Baruch" == Baruch Siach <baruch at tkos.co.il> writes:

 > Hi Peter,
 > Peter Korsgaard writes:
 >> Let the option depend on linux-pam as pam has quite some dependencies and
 >> this is unlikely to be used if pam isn't explicitly enabled.
 >> 
 >> Signed-off-by: Peter Korsgaard <peter at korsgaard.com>
 >> ---
 >> package/lighttpd/Config.in   | 7 +++++++
 >> package/lighttpd/lighttpd.mk | 8 +++++++-
 >> 2 files changed, 14 insertions(+), 1 deletion(-)
 >> 
 >> diff --git a/package/lighttpd/Config.in b/package/lighttpd/Config.in
 >> index 383c10f70f..0f37422aea 100644
 >> --- a/package/lighttpd/Config.in
 >> +++ b/package/lighttpd/Config.in
 >> @@ -26,6 +26,13 @@ config BR2_PACKAGE_LIGHTTPD_OPENSSL
 >> help
 >> Enable OpenSSL support for lighttpd.
 >> 
 >> +config BR2_PACKAGE_LIGHTTPD_PAM
 >> +       bool "pam authentication support"
 >> +       depends on BR2_PACKAGE_LINUX_PAM
 >> +       default y
 >> +       help
 >> +         Enable PAM authentication support for lighttpd.
 >> +

 > Why do we need a config option? What is the size impact of the lighttpd
 > PAM support?

Just for consistency with the other BR2_PACKAGE_LIGHTTPD_* options. I
would also prefer to use automatic dependencies like we do elsewhere,
but as we are explicitly handling the other optional dependencies it
seemed more sensible to do it for pam as well.

-- 
Bye, Peter Korsgaard