On Thu, 17 Jul 2014, Michael Osipov wrote:

> case 'l': /* --negotiate */
> if(toggle) {
> - if(curlinfo->features & CURL_VERSION_GSSNEGOTIATE)
> - config->authtype |= CURLAUTH_GSSNEGOTIATE;
> + if(curlinfo->features & CURL_VERSION_GSSAPI ||
> + curlinfo->features & CURL_VERSION_SSPI)
> + config->authtype |= CURLAUTH_NEGOTIATE;

It feels strange that you want this done in patch 4 and then in patch 5 you
again change this very code again. Can you squash them instead to a single
change?

Is there a point in using both terms Negotiate and SPNEGO in symbol names if
SPNEGO is required to do Negotiate?

-- 
  / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html