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: More passwd/group patches


On Sun, Nov 24, 2002 at 11:28:17AM -0500, Pierre A. Humblet wrote:
> At 05:08 PM 11/24/2002 +0100, Corinna Vinschen wrote:
> >The appropriate changes to sec_acl.cc would collide with your patch
> >so I'd like to ask you if you want me to make the necessary changes
> >to comply with Solaris first and then to send a revised patch, or
> >if you want to incorporate these changes into your patch, too?
> 
> Corinna,
> 
> Between the two proposals I prefer the first (you make the change first).
> The don't like the second much, because there is a possibility of 
> misunderstanding.
> There is also a third possibility: apply the patch and then fix it to
> match solaris. That may minimize the overall amount of work and 
> reviewing.

Ok, I'm going to apply your patch and then rework it as I think fit.

> I have been working on the home directory problem but can't reproduce 
> it. 
> I don't see anything wrong with his passwd file. Do you?

There isn't.  It looks like the files created by mkpasswd/mkgroup.

I'm pretty sure it's related to setting myself->gid to the new
DEFAULT_GID 401 in internal_getlogin.  It's perhaps a problem of the
execution order.  However, I didn't look into the code so far since
I'm busy with the sec_acl stuff.

Corinna


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


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