RE: NTLM authentication in CVS
> 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
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?
Re: NTLM authentication in CVS
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
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
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
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
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.