[PATCH] Fix minor breakage on Cygwin: auth-passwd.c andsession.c
Darren Tucker
dtucker at zip.com.au
Mon Aug 11 21:25:44 EST 2003
Corinna Vinschen wrote:
> On Mon, Aug 04, 2003 at 12:32:36AM -0500, Ben Lindstrom wrote:
> > > > There is enough of this junk laying around in different files it really
> > > > should be put in an openbsd-compat/port-cygwin.h =)
>
> Well, this isn't more "junk" than the extra stuff necessary for any
> other OS.
Er, I think the smiley at the end implied that "junk" meant "unnecessary
platform specific code".
As auth-passwd.c must be periodically synced with the OpenBSD tree,
keeping platform specific code out of it where possible makes that
easier. Compare auth-passwd.c from 3.6.1p2, OpenBSD and -current (after
Ben's cleanup).
> > However, yes. The code should go to bsd-cygwin_util.h unless there is
> > some overriding reason not to... Corinna is there a reason we shouldn't
> > make this change?
>
> No. But why is it still in xcrypt.c? It's in xcrypt.c just because it
> doesn't include openbsd-compat.h. Is there any good reason not to do that?
It's included indirectly:
xcrypt.c -> includes.h -> openbsd-compat.h -> bsd-cygwin_util.h
Now the code in question is in bsd-cygwin_util.h it doesn't need to be in
xcrypt.c any more either. I'll take it out.
--
Darren Tucker (dtucker at zip.com.au)
GPG key 8FF4FA69 / D9A3 86E9 7EEE AF4B B2D4 37C9 C982 80C7 8FF4 FA69
Good judgement comes with experience. Unfortunately, the experience
usually comes from bad judgement.
More information about the openssh-unix-dev
mailing list