[Kde-games-devel] Review Request 114974: Add example player to konquest when built in non-release mode

[Alexander Schuch]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114974/#review47752
-----------------------------------------------------------

Ship it!


Okay then. Looks fine. And once we know more about CMake, we simply improve it. :)

- Alexander Schuch


On Jan. 19, 2014, 11:24 p.m., Nemanja Hirsl wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114974/
> -----------------------------------------------------------
> 
> (Updated Jan. 19, 2014, 11:24 p.m.)
> 
> 
> Review request for KDE Games and Alexander Schuch.
> 
> 
> Repository: konquest
> 
> 
> Description
> -------
> 
> Just realized that we didn't include example player in (all) debug build types.
> http://techbase.kde.org/Development/Tutorials/Debugging/Debugging_symbols#And_make.3F
> 
> Alternative is to set it just for some build types, like:
> - for debug:  set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -DKONQUEST_DEBUG")
> - for debugfull set(CMAKE_CXX_FLAGS_DEBUG_FULL "${CMAKE_CXX_FLAGS_DEBUG_FULL} -DKONQUEST_DEBUGFULL")
> but I'm not sure that we want to do that.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt a20ce0b 
>   dialogs/newgamedlg.cc 0485cdd 
> 
> Diff: https://git.reviewboard.kde.org/r/114974/diff/
> 
> 
> Testing
> -------
> 
> Checked with release, debug and debugfull build types. Example player is not shown in release.
> 
> 
> Thanks,
> 
> Nemanja Hirsl
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20140120/b929a70a/attachment.html>