[Buildroot] [PATCH 1/1] libpam-tacplus: new package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Thu Aug 20 08:02:57 UTC 2015
Ryan, Giovanni,
On Wed, 19 Aug 2015 17:06:00 -0500, Ryan Barnett wrote:
> > diff --git a/package/libpam-tacplus/Config.in b/package/libpam-tacplus/Config.in
> > new file mode 100644
> > index 0000000..88d10ff
> > --- /dev/null
> > +++ b/package/libpam-tacplus/Config.in
> > @@ -0,0 +1,5 @@
> > + config BR2_PACKAGE_LIBPAM_TACPLUS
> > + bool "libpam-tacplus"
> > + help
> > + Lib needed to use pam-tacplus
>
> Would spell 'Library' out completely. It would also be good if you
> could link to the projects homepage as well.
Agreed, but I believe the description could also be improved by
copy/pasting more from the upstream website:
TACACS+ protocol client library and PAM module in C. This PAM
module support authentication, authorization (account
management) and accounting (session management)performed
using TACACS+ protocol designed by Cisco.
> > +LIBPAM_TACPLUS_INSTALL_STAGING = YES
> > +LIBPAM_TACPLUS_AUTORECONF = YES
Please add a comment above this line:
# Fetching from github, we need to generate the configure script
> > +LIBPAM_TACPLUS_CONF_OPTS = CC="$(TARGET_CC)" LD="$(TARGET_LD)"
>
> I'm not sure if this is needed but I could definitely be wrong since I
> haven't had a chance to dig into the package. It seems like this is
> something that buildroot's autotools package infrastructure should
> take care of passing into in order to ensure that it using the correct
> compiler.
The package is using autoconf, but not automake: it uses a hand-written
Makefile. Due to this, passing some variable at build time might be
required. However, it would be useful to try something like:
LIBPAM_TACPLUS_CONF_ENV = $(TARGET_CONFIGURE_OPTS)
instead.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list