RE: NTLM authentication in CVS

2005-04-08 Thread Herold Heiko
 From: Hrvoje Niksic [mailto:[EMAIL PROTECTED]
 
  3) As expected msvc still throws compiler error on http.c 

 1. Is it possible to only lessen the optimization level, not entirely
remove optimization?

Fooled around a bit, didn't accomplish anything. However it is enough
disabling the optimization for those two functions, only.
 
 2. Is it possible to test for the offending version of the compiler,
and only then remove optimization?

Since Tobias (thanks!) confirmed v13 works yes.
The enclosed patch disables for _MSC_VER=1200, if other reports should come
in maybe we can restrict the test further.

Heiko

-- 
-- PREVINET S.p.A. www.previnet.it
-- Heiko Herold [EMAIL PROTECTED] [EMAIL PROTECTED]
-- +39-041-5907073 ph
-- +39-041-5907472 fax



20050408.diff
Description: Binary data


Re: NTLM authentication in CVS

2005-04-07 Thread Daniel Stenberg
On Thu, 7 Apr 2005, Hrvoje Niksic wrote:
If someone has access to an NTLM-authorizing web server, please try it
out with Wget and let us know how it goes.
It should be noted that for NTLM, you can specify the domain name in the user 
name field as:

 domain\user or domain/user
--
 -=- Daniel Stenberg -=- http://daniel.haxx.se -=-
  ech`echo xiun|tr nu oc|sed 'sx\([sx]\)\([xoi]\)xo un\2\1 is xg'`ol


RE: NTLM authentication in CVS

2005-04-07 Thread Herold Heiko
back after a while

In order to compile current cvs with msvc 3 patches are needed (enclosed):

1)
mswindows.c(118) : warning C4005: 'OVERFLOW' : macro redefinition
C:\PROGRA~1\MICROS~2\VC98\INCLUDE\math.h(415) : see previous
definition of 'OVERFLOW'
mswindows.c(119) : warning C4005: 'UNDERFLOW' : macro redefinition
C:\PROGRA~1\MICROS~2\VC98\INCLUDE\math.h(416) : see previous
definition of 'UNDERFLOW'

(simple rename of course solves)

2) add http-ntlm.c to windows Makefiles (enclosed Borland/Watcom/Mingw,
tested msvc only)

3) As expected msvc still throws compiler error on http.c and retr.c, (bad)
workaround: disable optimization. Anybody with a cl.exe newer than
Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 12.00.8804 for 80x86
can comment if this is needed with newer versions, too ?

Binary for evaluation available at http://xoomer.virgilio.it/hherold as
usual.

I still can't access the sunsite ftp repository, maybe somebody who can
should at least remove/update the 00Readme.txt and index.html (extremely
outdated), probably throw away the pletora of old development binaries and
sources (everything named wgetmmdd[sb].zip).

Beside that, I'm still very short on time and can barely follow development,
in future most probably the situation will be worse (due to personal
issues). If there would be anybody else capable and volunteering to release
msvc binaries somewhat regulary I'd be more than glad.

Heiko

-- 
-- PREVINET S.p.A. www.previnet.it
-- Heiko Herold [EMAIL PROTECTED] [EMAIL PROTECTED]
-- +39-041-5907073 ph
-- +39-041-5907472 fax

 -Original Message-
 From: Hrvoje Niksic [mailto:[EMAIL PROTECTED]
 Sent: Thursday, April 07, 2005 4:01 PM
 To: wget@sunsite.dk
 Subject: NTLM authentication in CVS
 
 
 As of two days ago, the NTLM authentication support is in CVS,
 although it has undergone very little testing.  (To quote Linus, If
 it compiles, it's perfect.)
 
 If someone has access to an NTLM-authorizing web server, please try it
 out with Wget and let us know how it goes.  Unfortunately, this
 version of Wget doesn't support NTLM for proxies[1], so you'll need a
 *server* that authorizes over NTLM.
 
 
 [1]
 The proxy limitation will hopefully be fixed after 1.10 is released.
 Fixing it for 1.10 would be too much work since it might require using
 HTTP 1.1.
 



20050407b.diff
Description: Binary data


20050407a.diff
Description: Binary data


20050407c.diff
Description: Binary data


Re: NTLM authentication in CVS

2005-04-07 Thread Tobias Tiederle
Herold Heiko schrieb:

3) As expected msvc still throws compiler error on http.c and retr.c, (bad)
workaround: disable optimization. Anybody with a cl.exe newer than
Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 12.00.8804 for 80x86
can comment if this is needed with newer versions, too ?

I'm using
Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 13.00.9466 for 80x86

the only noteable output while compiling (your other two patches
applied) is:
\Vc7\PlatformSDK\Include\WinSock.h(689) : warning C4005: 'NO_ADDRESS' :
macro redefinition
host.c(59) : see previous definition of 'NO_ADDRESS'
http.c(514) : warning C4090: 'function' : different 'const' qualifiers
http.c(532) : warning C4090: 'function' : different 'const' qualifiers
http.c(710) : warning C4090: 'function' : different 'const' qualifiers

Tobias


Re: NTLM authentication in CVS

2005-04-07 Thread Hrvoje Niksic
Herold Heiko [EMAIL PROTECTED] writes:

 In order to compile current cvs with msvc 3 patches are needed
 (enclosed):

Thanks for testing!

 1)
 mswindows.c(118) : warning C4005: 'OVERFLOW' : macro redefinition
 C:\PROGRA~1\MICROS~2\VC98\INCLUDE\math.h(415) : see previous
 definition of 'OVERFLOW'
 mswindows.c(119) : warning C4005: 'UNDERFLOW' : macro redefinition
 C:\PROGRA~1\MICROS~2\VC98\INCLUDE\math.h(416) : see previous
 definition of 'UNDERFLOW'

Fixed now.

 (simple rename of course solves)

 2) add http-ntlm.c to windows Makefiles (enclosed Borland/Watcom/Mingw,
 tested msvc only)

Note that http-ntlm should only be used if SSL is available.  How does
one do that in the MSVC Makefile?  We should use the same technique
that is used to compile SSL source files only when SSL is used.

 3) As expected msvc still throws compiler error on http.c and retr.c, (bad)
 workaround: disable optimization. Anybody with a cl.exe newer than
 Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 12.00.8804 for 80x86
 can comment if this is needed with newer versions, too ?

It seems that newer versions work correctly.  Two questions, though:

1. Is it possible to only lessen the optimization level, not entirely
   remove optimization?

2. Is it possible to test for the offending version of the compiler,
   and only then remove optimization?

 Beside that, I'm still very short on time and can barely follow
 development, in future most probably the situation will be worse
 (due to personal issues). If there would be anybody else capable and
 volunteering to release msvc binaries somewhat regulary I'd be more
 than glad.

Your contributions are much appreciated, thanks!  I'm sure someone
with more time will step up to do the Windows binaries.  (Now that
Wget compiles with the free CLI version of the Borland compiler, this
is easier than ever.)


Re: NTLM authentication in CVS

2005-04-07 Thread Hrvoje Niksic
Tobias Tiederle [EMAIL PROTECTED] writes:

 the only noteable output while compiling (your other two patches
 applied) is:
 \Vc7\PlatformSDK\Include\WinSock.h(689) : warning C4005: 'NO_ADDRESS' :
 macro redefinition
 host.c(59) : see previous definition of 'NO_ADDRESS'

I've now fixed this.

 http.c(514) : warning C4090: 'function' : different 'const' qualifiers
 http.c(532) : warning C4090: 'function' : different 'const' qualifiers
 http.c(710) : warning C4090: 'function' : different 'const' qualifiers

I don't understand these warnings.  According to the documentation
available on the web, the C4090 warning occurs when the const
qualifier is being lost, for instance by supplying the const pointer
to a function that expects a non-const pointer.  However, that does
not happen in any of the cited source lines.

Can someone knowledgable with MSVC explain the warnings?