Andrew Dunstan schrieb:
> Magnus Hagander wrote:
>> Heh. Maybe not dead, but certainly not really alive either ;-) Given the
>> evidence in your patch that clearly 8.3 isn't quite up to speed on
>> cygwin, and nobody has really noticed until now.
>>
>
> AIUI, only the gssapi stuff is broken. Most users are not likely to want
> it on Cygwin I should think. (And plenty of distros are still on 8.2 and
> earlier, anyway).
Well, native windows users is a very nice to have. Actually a killer
feature.
> What would be nice would be for Reini to set up a Cygwin buildfarm
> member that uses all the switches that the Cygwin distro wants to use.
Without ENABLE_SSPI I just need --enable-nls --with-CXX --with-openssl --with-perl --with-python --with-libxml
--with-libxslt--with-ldap
and these build out of the box.
>> I also notice this in auth.c:
>> +#ifdef·__CYGWIN__
>> +#define·WIN32
>> +#endif
>>
>> What is the need to change this for just one file? Seems very fragile -
>> the rest of our codebase assumes WIN32 != CYGWIN, and I think we should
>> keep that consistent.
>
> Right. We have had significant grief from this in the past, and don't
> need to return there. If we need it to get correct behaviour from some
> header file, then it needs to be heavily commented and localised. But I
> bet there are other ways to get the right result - that's what we've
> tended to find in the past.
Ok, I copy then the required lines from WIN32.
This was the shortest patch I could come up with and it
worked nice for the auth backend, with SSPI enabled.
--
Reini