This is the mail archive of the
cygwin-patches@cygwin.com
mailing list for the Cygwin project.
Re: [PATCH]: Check modification time on /etc/passwd and /etc/group
On Sun, Aug 05, 2001 at 09:38:17PM -0400, Christopher Faylor wrote:
> On Sun, Aug 05, 2001 at 09:56:52PM +0200, Corinna Vinschen wrote:
> >On Sun, Aug 05, 2001 at 02:17:18PM -0400, Christopher Faylor wrote:
> >> On Sun, Aug 05, 2001 at 07:59:59PM +0200, Corinna Vinschen wrote:
> >> >On Sun, Aug 05, 2001 at 12:29:54PM -0400, Christopher Faylor wrote:
> >> >>So, is it worthwhile to make this change? We might be adding some
> >> >>overhead for an infrequent case.
> >> >
> >> >Hmm, that's a good question. I would like to see that handled
> >> >correctly w/o having to stop and start my services again. And it's
> >> >actually no infrequent case for sysadmins to add users.
> >>
> >> I wasn't talking about your change to automatically update /etc/passwd.
> >> I think that is valuable.
> >>
> >> I was more concerned about spending time to optimize /etc/passwd accesses
> >> not actually optimizing anything.
> >
> >So you would prefer something silmilar to my first solution which
> >actually calls `stat()'?
>
> No. Your last solution was fine. Calculating the windows path once is,
> IMO, fine, too.
>
> I was referring to the goal of either putting the passwd info in the cygheap
> or avoiding the use of fopen/fgets/whatever.
I see. However, my patch isn't that useful in the current state since
it doesn't free the old passwd datastructures and buffers which must
still be added.
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Developer mailto:cygwin@cygwin.com
Red Hat, Inc.