[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#502782: [libpam-ldap] "let's" in extended description



(Try again, including reasoning and CCs)

Christian Perrier wrote:
> As I'm in the process of NMU'ing the package for l10n we could use
> this opportunity to review the package description.
> 
> Description: Pluggable Authentication Module allowing LDAP interfaces

"Interfaces" is an odd choice of word - why plural?

>  This module let's you use you LDAP server to authenticate users with
>  programs that utilize PAM.

s/let's/lets/ and s/you/your/, but get rid of the second person 
pronouns (and "utilize" while I'm at it).  "With" feels like the
wrong preposition, too; it's easier just to start from scratch.

>                             If used along with libnss-ldap, you can
>  replace your entire flat file (/etc/*) structure or NIS with LDAP.

This dangling participle structure implies it's "you" that's "used
along with libnss-ldap".  Then "replace [complex phrase] with LDAP"
is confusing ("...or NIS with LDAP"?), and LDAP certainly doesn't
replace /etc/*.

So I'd suggest:

  Description: Pluggable Authentication Module for LDAP
   This package provides an interface between an LDAP server and the PAM
   user authentication system. Using it along with libnss-ldap allows LDAP
   to replace the entire /etc/passwd system, or alternatives such as NIS.

-- 
JBR	with qualifications in linguistics, experience as a Debian
	sysadmin, and probably no clue about this particular package


Reply to: