realXtend / tundra

realXtend Tundra SDK, a 3D virtual world application platform.
www.realxtend.org
Apache License 2.0
84 stars 70 forks source link

Not possible to have both Debug and Release(/RelWithDebInfo/MinSizeRel) working builds at the same time #740

Open Stinkfist0 opened 10 years ago

Stinkfist0 commented 10 years ago

After running both BuildDeps_Debug and BuildDeps_RelWithDebInfo and building Tundra using Debug configuration, Tundrad.exe as will crash due to having wrong and incompatible qjson.dll version (RelWithDebInfo) in the /bin directory. As a solution, Debug version of the qjson.dll must have some kind of debug postfix.

jonnenauha commented 10 years ago

As talked on IRC we need to modify the projects cmake config. We are currently cloning my fork of the repo. These guys don't seem to be very interested in Windows so I'm sure we can get these trivial changes to upstream also.

Anyone who wants to tackle this send me a pull request https://github.com/jonnenauha/qjson, or I guess directly to the original repo and let me know so I can update my fork.

This is the line that needs to be added https://github.com/realXtend/naali/blob/tundra2/cmake/ModuleSetup.cmake#L163 I would wrap it with if (MSVC) in cmake so its more likely to end up upstream.