playground/games/picmi moved to KDE Review

[Jakob Gruber]

On 07/24/2012 11:35 PM, Laszlo Papp wrote:
>> Does the game work on Windows?
> 
> I have just put some effort into testing this, but it is quite broken.
> I would like to see improvements in this front since Windows is an
> important platform for Qt and KDE. I am here for helping with that, if
> needed.

Thanks for testing, I appreciate the time spent.

I'd like to clarify that all of the points mentioned in this mail
concern libkdegames (and compilation thereof on Windows) and not picmi
in particular, therefore I'm forwarding this to kde-games-devel.

> The issues so far (until I found enough to not bother with finding
> more as of now) in kdegames (dependency):
> 
> 1) FindSndFile.cmake is broken. It does not find the installed sndfile
> on my system which it should. I would suggest using the one we used in
> alure, gluon, QtOpenAL back then and so forth.
> 
> 2) http://websvn.kde.org/trunk/KDE/kdegames/libkdegames/CMakeLists.txt?view=markup
> -> file(RELATIVE_PATH CONF_REL_INCLUDE_DIR
>     "${DATA_INSTALL_DIR}/cmake/modules"
>     "${INCLUDE_INSTALL_DIR}")
> 
> This is not going to work since you need to use absolute paths, if I
> am not mistaken. ${CMAKE_INSTALL_PREFIX} will help you out about that.
> 
> 3) http://websvn.kde.org/trunk/KDE/kdegames/libkdegames/audio/kgsound-openal.cpp?view=markup:
> #include <sndfile.hh> //TODO: use Phonon instead of libsndfile for
> decoding -> You should use "sndfile.h"
> 
> 4) You use "class Private" here:
> http://websvn.kde.org/trunk/KDE/kdegames/libkdegames/audio/kgsound.h?view=markup
> and then "struct Foobar::Private" here:
> http://websvn.kde.org/trunk/KDE/kdegames/libkdegames/audio/kgsound-openal.cpp?view=markup
> 
> ... after this point I gave up for now. Please fix the issues in
> kdegames along with the responsible person(s), and I will make check
> further checks. The game, and libkdegames has not clearly been tested
> properly with openal and sndfile. That may also be due to the fact,
> the FindSndFile.cmake is broken.
> 
> Unfortunately, phonon master also had issues, so we just recommend
> using the last phonon stable release in certain cases.
> 
> Let me know, if you need help with all these. I can also commit the
> fixes myself, if it is accepted by whoever the maintainer is.
> 
> Best Regards,
> Laszlo Papp
>