[Buildroot] [PATCH v5] package/pamtester: new package

Carlos Santos casantos at datacom.com.br
Tue Mar 19 01:35:27 UTC 2019


> From: "Peter Korsgaard" <peter at korsgaard.com>
> To: "Matthew Weber" <matthew.weber at rockwellcollins.com>
> Cc: "buildroot" <buildroot at buildroot.org>
> Sent: Segunda-feira, 4 de fevereiro de 2019 15:07:12
> Subject: Re: [Buildroot] [PATCH v5] package/pamtester: new package

>>>>>> "Matt" == Matt Weber <matthew.weber at rockwellcollins.com> writes:
> 
> > From: Carlos Santos <casantos at datacom.com.br>
> > Utility for testing pluggable authentication modules (PAM) facility.
> 
> > While specifically designed to help PAM module authors to test their
> > modules, that might also be handy for system administrators interested
> > in building a centralised authentication system using common standards
> > such as NIS, SASL and LDAP.
> 
> > One patch was included from the Debian bugtracker which fixes a
> > spelling error.
> 
> I still don't understand why we need to carry such patch.

Because I'm a maniac and the misspelling puts me nervous.

> > Signed-off-by: Carlos Santos <casantos at datacom.com.br>
> > Signed-off-by: Matthew Weber <matthew.weber at rockwellcollins.com>
> > ---
> > Changes
> > v4 -> v5
> >  - fixed title of patch to be package/
> >  - Added signed-offs in Debian patch
> >  - Added license file hash
> >  - Spelling of pamtester in kconfig comment
> >  - DEVELOPERS entry needed fixup
> > ---
> 
> > +++ b/package/pamtester/Config.in
> > @@ -0,0 +1,15 @@
> > +config BR2_PACKAGE_PAMTESTER
> > +	bool "pamtester"
> > +	depends on BR2_PACKAGE_LINUX_PAM
> > +	help
> > +	  A tiny utility for testing pluggable authentication modules
> > +	  (PAM) facility. While specifically designed to help PAM module
> > +	  authors to test their modules, that might also be handy for
> > +	  system administrators interested in building a centralised
> > +	  authentication system using common standards such as NIS, SASL
> > +	  and LDAP.
> > +
> > +	  http://pamtester.sourceforge.net/
> > +
> > +comment "pamtester depends on linux-pam"
> > +	depends on !BR2_PACKAGE_LINUX_PAM
> 
> I'm not sure this comment is really helpful, but OK.
> 
> Committed with the spelling fix patch dropped, thanks.

I accept that as a sad fact of life. Thanks. ;-)

-- 
Carlos Santos (Casantos) - DATACOM, P&D


More information about the buildroot mailing list