[Pkg-shadow-devel] [PATCH v2 5/5] Allow creating shadow files, when missing, via an option.
Mike Frysinger
vapier at gentoo.org
Mon Apr 6 16:47:37 UTC 2015
On 06 Apr 2015 10:24, Dimitri John Ledkov wrote:
> On 5 April 2015 at 03:50, Mike Frysinger wrote:
> > On 01 Apr 2015 13:43, Dimitri John Ledkov wrote:
> >> +#
> >> +# Create shadow(5) and gshadow(5) files, when operations are invoked
> >> +# that could modify those. E.g. passwd/useradd/userdel/usermod etc.
> >> +# If this option is not set, shadow(5)/gshadow(5) files are updated,
> >> +# only if already present on disk.
> >> +#
> >> +#
> >> +#CREATE_SHADOW yes
> >> ...
> >> bool sgr_file_present (void)
> >> {
> >> + if (getdef_bool ("CREATE_SHADOW"))
> >> + return true;
> >> return commonio_present (&gshadow_db);
> >> }
> >
> > looks to me like you're changing the default ? it is now disabled by default
> > instead of enabled by default.
> >
>
> All options are disabled by default. The bit in the config file is
> commented out, thus getdef_bool will return false by default. Thus the
> presence of gsahdow_db will remain as the default condition as to
> whether to use gshadow file or not. (same for shadow)
>
> Should first hunk then instead be:
>
> +#CREATE_SHADOW no
>
> ? And the manpage update, once this is settled.
what i mean is that today, the behavior is as if you set:
CREATE_SHADOW yes
but after your patch, the behavior is:
CREATE_SHADOW no
i'm saying that the out-of-box behavior shouldn't change. i guess that means
you want to add it uncommented:
CREATE_SHADOW yes
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-shadow-devel/attachments/20150406/cabd8dc9/attachment.sig>
More information about the Pkg-shadow-devel
mailing list