Hi There, This would be related to the open-ssl thing I did... Basically rather than rewriting a bunch of wininet code, I wrote a layer around the winsock stuff so that it can easily switch between ssl and non-ssl sockets.. The odd thing is in my WineX tree (where I do my development) that fixme isn't in InternetReadFile - theres now only one, and thats in InterneWriteFile... Mike McCormack - is it possible this file (internet.c) didn't get merged properly? Mike Hearn - check out the function 'InternetReadFile' at this URL and compare against your post-patched winehq version. http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/winex/wine/dlls/wininet/internet.c?rev=1.3&content-type=text/vnd.viewcvs-markup *few minutes later* ok, so I checked out winehq cvs and applied the patch.. Mike McCormack - it looks like you merged the contents of InternetWriteFile into both InternetWriteFile and InternetReadFile.. I may be wrong (I just woke up).. The FIXME in InternetWriteFile has no bad effect, things will still work (so long as you're not using HTTPS) - but in InternetReadFile, I have my doubts. Cheers. David On Mon, 2 Jun 2003 23:32, Mike Hearn wrote: > Well, I'm spending my fun afternoon trying to get IE to install in Wine > (it breaks in a new and interesting way every time I try this). > > Before the patch, it didn't work, couldn't contact download sites. After > the patch, it still doesn't work, and I get this fixme: > > fixme:wininet:InternetReadFile This shouldn't be here! We don't support > this kind of connection anymore. Must use NETCON functions, especially > if using SSL > > Which seems a bit of a bummer, if IE6 setup needs that type of > connection I think we should still support it really...... > > I think the problem might be lack of proxy workingness. Am investigating > now. Would be nice to lift proxy details from the http_proxy variable in > future.... > > One last thing, the patch seemed to have a lot of whitespace no-ops, > running diff with -bB should eliminate some. > > thanks -mike -- David Hammerton TransGaming Technologies david@transgaming.com http://www.transgaming.com/ *Let the games begin*
