[Bug 3771] New Visual Studio 2005 project file

[bugzilla-daemon at icculus.org]

http://bugzilla.icculus.org/show_bug.cgi?id=3771





------- Comment #20 from monk at rq3.com  2009-02-10 02:25 EDT -------
(In reply to comment #18)
> (In reply to comment #17)
> However, reading the diff I still see references to libcurl.lib. Since this

> See lines similar to:
> -                               AdditionalDependencies="user32.lib advapi32.lib
> winmm.lib wsock32.lib openal32.lib SDLmain.lib SDL.lib OpenGL32.lib
> libcurl.lib"
> +                               AdditionalDependencies="user32.lib advapi32.lib
> winmm.lib ws2_32.lib openal32.lib SDLmain.lib SDL.lib OpenGL32.lib libcurl.lib"
> As you're defining USE_CURL_DLOPEN you do not need to link with libcurl.lib, so
> this should be removed from these lines.


Looking at my project file, I do NOT use libcurl.lib, so that diff must be
really jacked up.

> Personally I prefer my patch to the USE_LOCAL_HEADERS constant rather than
> simply adding the \code\libcurl\... etc directories onto the include paths
> since I feel this is more correct. Unless ioq3 is intended to only use the
> local SDL / curl headers, I think this constant should be respected. Instead,
> you're making sure a "local" directory is in the search path, which means it
> will always use the local headers in preference to system, regardless of that
> constant. Compare how the jpeg includes are redirected to the local path with
> that constant set. However I do not particularly care - all I'm interested in
> is the SVN head compiles. 

This may be intended as ioq3 seems to keep a certain set of tweaked libraries
like an old version of zlib and some "fixed" version of SDL.  Also since ioq3
includes these in the source tree, that seems to reinforce that intent?  The
only outlier seems to be the SDL library which on the Windows platform you have
to get yourself.  Though there's a fixed sdl.dll that you need to download
yourself (it'll be included in 1.36) to play ioq3, which is odd.  I'm guessing
SDL is just too big and unwieldy to include in the ioq3 source tree for that
fix?    I will attach the entire XML project and see if that works.  Just be
aware you'll need the config.h also linked as bug 3857.


-- 
Configure bugmail: http://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.