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

Re: Bug#654848: [RFR] templates://yubico-pam/{libpam-yubico.templates}



Quoting Simon Josefsson (simon@josefsson.org):
> Many thanks for your review, Christian.  I'm fine with all your changes.
> I also learned that 'Description:' fields now apparently are prefered to
> start lower-cased, which was news to me.  Possibly the Description field
> becomes even more clear like this though:
> 
> OLD:
> Description: two-factor password+OTP (YubiKey) PAM module
> 
> NEW:
> Description: two-factor password and YubiKey OTP (one-time password) PAM module
> 
> although I don't recall what the exact max line length is.


A bit shorter (76 chars is strongly suggested).

I adopte this without the parenthesis as "one-time password" is
expanded in the long description.

Otherwise, I adopted all suggestions by Justin.

What would be nice is to have a comment of yours about the debconf
note (the second debconf template and Justin's suggestion to add a
paragraph in the first one that would mention that the PAM module is
disabled....and therefore making the second template useless).


Attachment: signature.asc
Description: Digital signature


Reply to: