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

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

On Wed, Jan 11, 2012 at 7:10 AM, Christian PERRIER <bubulle@debian.org> wrote:
> 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).

I would like to get rid of the note too.

Personally, I sort of expect things I "apt-get install" to be enabled
by default, but that was not possible with this PAM module because you
could accidentally lock yourself out of your system if you installed
it and did not finish the configuration (I got bug reports about that
when this was a PPA).

I guess that is why I thought a debconf note was warranted, but since
you said that kind of notice usually goes into README.Debian, I am all
for removing the debconf note.

Regarding Justins note :

> By the way, I've just noticed (too late for this patch) that
>  Homepage: http://code.google.com/p/yubico-pam/
> should now point towards https://github.com/Yubico/yubico-pam.

I'd say the project website is at code.google.com, although the source
and wiki is currently at github.com. Unless the Homepage: is supposed
to point at where you get the source, I would think it is more
appropriate to keep Homepage: code.google.com.

Thank you both for the excellent review!


Reply to: