On Wed, May 25, Tomas Mraz wrote:
On Wed, 2011-05-25 at 15:37 +0200, Thorsten Kukuk wrote:
> Hi,
>
> I found out that pam_env.so has a "debug" option, which
> is described in the manual page, but the variable is never
> used. So it's a NOP.
> Should we remove it? I would suggest to remove it at least
> from the documentation.
>
> Else pam_timestamp as an used variable, Ok to remove it?
OK from me for both removing the unused variable in pam_timestamp and
the debug option from the pam_env code and man page - there is no need
to keep dead code.
I was more afraid of compatibility, but since a wrong option
will only lead to an error message, but not an abort, I will
remove it completly.
Thorsten
--
Thorsten Kukuk, Project Manager/Release Manager SLES
SUSE LINUX Products GmbH, Maxfeldstr. 5, D-90409 Nuernberg
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg)