This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: mkpasswd & mkgroup


On Mon, Feb 24, 2003 at 11:29:15PM -0500, Pierre A. Humblet wrote:
> Corinna,
> 
> Earlier I have added a -c switch to mkpasswd & mkgroup. 
> It prints the current user (if a domain account), without 
> contacting the PDC. 
> That was meant for use in postinstall scripts.
> 
> However until setup.exe is upgraded, only mkpasswd -l will
> be run by passwd-grp.bat, which causes recurring problems with
> domain users. 
> 
> Thus, at least until setup.exe is upgraded, I suggest that
> -c be implied by -l, unless -d is also specified.
> This is implemented by the attached patch.
> It pretty much insure that the main users will have proper entries.

What is the impact for setup.exe?  In my eyes the change is fine
and could be kept this way.  It wouldn't require any change to
setup.exe and *especially* it would'n be good to change mkpasswd
and mkgroup for a short period of time just to revert the patch
a few days (I'm an optimist) later.

So, either we just keep mkpasswd and mkgroup now as it is, or we
make sure that setup.exe won't rely on the -c flag in the next
version but just uses -l as today.

> Also, to support arbitrary uid's on Win95, mkpasswd prints both
> a default line with uid 500, and a line for the current user 
> with a pseudorandom uid. Other users can be added with -u.
> Cygwin uses the default line for users that do not have an entry.

That's fine.  Regardless of the above, we could apply this one.
Feel free to commit that change.

Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin at cygwin dot com
Red Hat, Inc.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]