Re: secur32: Add a warning if ntlm_auth is not found or in a wrongversion.

[Dmitry Timoshkov]

"Kai Blin" <kai.blin(a)gmail.com> wrote:
> +    else
> +    {
> +        WARN("ntlm_auth not found, or in an outdated version.\n" \
> +             "Make sure you are using ntlm_auth =< 3.x and it is in your path.\n");
> +    }

If the message is supposed to be user visible (otherwise why to bother
with wording?) it would be better to use either FIXME or MESSAGE.

Also "=<" is not really understandable in this context. Did you mean
that a version 3.x or better is required (then >= 3.x), or <= 3.x (then
the word "outdated" is misleading)?

-- 
Dmitry.

secur32: Add a warning if ntlm_auth is not found or in a wrongversion.

[Kai Blin]

On Sunday 17 September 2006 09:54, Dmitry Timoshkov wrote:
> "Kai Blin" <kai.blin(a)gmail.com> wrote:
> > +    else
> > +    {
> > +        WARN("ntlm_auth not found, or in an outdated version.\n" \
> > +             "Make sure you are using ntlm_auth =< 3.x and it is in your
> > path.\n"); +    }
>
> If the message is supposed to be user visible (otherwise why to bother
> with wording?) it would be better to use either FIXME or MESSAGE.

I put it in so we would be able to tell in a trace that ntlm_auth wasn't 
there. I guess we can make that visible  to the user.
> Also "=<" is not really understandable in this context. Did you mean
> that a version 3.x or better is required (then >= 3.x), or <= 3.x (then
> the word "outdated" is misleading)?

Yeah, good catch.. I must have been thinking in reverse polish notation or 
something. I'll fix that.

Kai

-- 
Kai Blin, <kai Dot blin At gmail Dot com>
WorldForge developer    http://www.worldforge.org/
Wine developer          http://wiki.winehq.org/KaiBlin/
--
Will code for cotton.