This is the mail archive of the
cygwin-patches@cygwin.com
mailing list for the Cygwin project.
RE: patch to mkpasswd.c - allows selection of specific user
- From: Mark Bradshaw <bradshaw at staff dot crosswalk dot com>
- To: "'cygwin-patches at cygwin dot com'" <cygwin-patches at cygwin dot com>
- Date: Mon, 19 Nov 2001 15:14:00 -0500
- Subject: RE: patch to mkpasswd.c - allows selection of specific user
Sure. I was working a bit with matthew to do both the -u and -c functions.
I'll follow up with him to see if he's still interested, and if not resubmit
the patch.
Mark
> -----Original Message-----
> From: Christopher Faylor [mailto:cgf@redhat.com]
> Sent: Monday, November 19, 2001 1:37 PM
> To: Mark Bradshaw
> Cc: cygwin-patches@cygwin.com
> Subject: Re: patch to mkpasswd.c - allows selection of specific user
>
>
> Mark,
> We have your assignment on file.
>
> Could you resubmit this patch?
>
> Thanks,
> cgf
>
> On Tue, Nov 13, 2001 at 08:09:10AM -0500, Mark Bradshaw wrote:
> >Sorry. I threw that in at the last. Thought it might be
> helpful for the
> >enduser to get a little feedback for that error. I'll fix it.
> >
> >> -----Original Message-----
> >> From: Christopher Faylor [mailto:cgf@redhat.com]
> >> Sent: Tuesday, November 13, 2001 12:22 AM
> >> To: cygwin-patches@cygwin.com
> >> Subject: Re: patch to mkpasswd.c - allows selection of
> specific user
> >>
> >>
> >> On Tue, Nov 13, 2001 at 04:02:32PM +1100, Mathew Boorman wrote:
> >> >Darn, now I'm told about Marks patch!
> >> >Anyway, onward...
> >> >
> >> >Mark Bradshaw:
> >> >
> >> >>@@ -135,6 +145,7 @@ enum_users (LPWSTR servername, int print
> >> >> default:
> >> >> fprintf (stderr, "NetUserEnum() failed with
> %ld\n", rc);
> >> >>+ if ( rc == 2221 ) printf("That user doesn't appear to
> >> exist.\n");
> >> >
> >> >The appropriate error codes are in <lmerr.h> around, I noted
> >> some were
> >> >slightly different in name though. I believe this message
> >> should go to
> >> >stderr anyway, otherwise you would end up with a polluted
> >> /etc/passwd
> >> >file.
> >>
> >> Oops. This points out a couple of problems that I didn't
> >> notice before.
> >>
> >> 1) Never use a raw number like the above, as Mathew has said.
> >>
> >> 2) This is not the correct format for an if statement. You
> >> aren't adhering
> >> to the GNU formatting conventions. Please use the
> >> formatting of the
> >> code that you are patching. This is good advice for
> >> whatever project
> >> you are on.
> >>
> >> Thanks,
> >> cgf
> >>
>
> --
> cgf@redhat.com Red Hat, Inc.
> http://sources.redhat.com/ http://www.redhat.com/
>