[Pkg-shadow-devel] Re: [Pkg-shadow-commits] r307 - trunk/debian/patches

Nicolas François nicolas.francois@centraliens.net
Mon, 27 Jun 2005 01:48:55 +0200


Hello,

On Sun, Jun 26, 2005 at 11:37:51PM +0000, Nicolas FRANCOIS wrote:
> Author: nekral-guest
> Date: 2005-06-26 23:37:50 +0000 (Sun, 26 Jun 2005)
> New Revision: 307
> 
> Added:
>    trunk/debian/patches/427_chage_expiry_0
> Modified:
>    trunk/debian/patches/series
> Log:
> Add a patch for #78961.
> Added: trunk/debian/patches/427_chage_expiry_0
> ===================================================================
> --- trunk/debian/patches/427_chage_expiry_0	2005-06-26 21:41:54 UTC (rev 306)
> +++ trunk/debian/patches/427_chage_expiry_0	2005-06-26 23:37:50 UTC (rev 307)
> @@ -0,0 +1,53 @@
> +Goal: Fix chage display for expiry fields set to 0.
> +Fixes: #78961
> +
> +Status wrt upstream: It should be forwarded to upstream.
> +
> +Note: On Debian, PAM deals unconsistently with these fields when they are
> +      set to 0. See #308229.

I've committed this patch, but there is a small issue for debian: su and
passwd don't deal the same way with a null expiry field. Also, Debian's
PAM has a patch which change the behavior of other fields when they are
set to 0.

So, shall we comment it in series until there is a clearer status for
#308229?
(There's no changelog entry for the moment)

Best Regards,
-- 
Nekral