[PATCH/cygwin]: Explicitely undefine _WIN32

Corinna Vinschen vinschen at redhat.com
Tue Mar 27 18:43:47 EST 2012


On Mar 19 10:28, Corinna Vinschen wrote:
> On Mar 17 09:07, Glenn Maynard wrote:
> > On Sat, Mar 17, 2012 at 5:10 AM, Corinna Vinschen <vinschen at redhat.com>wrote:
> > 
> > > +/* Make sure _WIN32 isn't defined later in the code, otherwise headers
> > > from
> > > +   other packages might get the wrong idea about the target system. */
> > > +#ifdef _WIN32
> > > +#undef _WIN32
> > > +#endif
> > >
> > 
> > #undef for an undefined macro should be a no-op, so the #ifdef should be
> > unnecessary.
> 
> You're right.  BUt it looks cleaner, doesn't it?  Anyway, whoever has
> checkin rights, please apply any one of the patches you like better.

Ping?  Could this please be applied for 6.0 so that I can release a
Kerberos-enabled version in the Cygwin distro?


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Project Co-Leader
Red Hat


More information about the openssh-unix-dev mailing list