[patch] Update bsd-cygwin_util files (was Re: Announce: OpenSSH 6.2 released)

Darren Tucker dtucker at zip.com.au
Mon Apr 1 12:41:59 EST 2013


On Wed, Mar 27, 2013 at 03:14:18PM +0100, Corinna Vinschen wrote:
[...]
> On second thought (and after some sleep), this is probably not the right
> thing to do.  Changing the type for UINT64 in the Windows headers just
> because of a single instance of clashing definitions in a POSIX file is
> pretty intrusive.  The potential side effects of this change are simply
> not worth it.
> 
> As a result I have now reworked the bsd-cygwin_util.* files in
> openbsd-compat.  The most important change is not to include the
> windows.h header at all.  The only two Windows definitions used are
> HANDLE and INVALID_HANDLE_VALUE.  I defined those in bsd-cygwin_util.h
> so there's no reason anymore to include windows headers.  This fixes
> the UINT64 issue as well, obviously.
> 
> Additionally I made the handling of open vs. binary_open a bit more
> clear (I think) by introducing a NO_BINARY_OPEN definition.
> 
> Is that ok to apply?

applied, thanks.

-- 
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