----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6656/ -----------------------------------------------------------
Review request for kde-windows. Summary ------- This is a patch for emerge to add a CMake recipe for libsdl-1.2 However, since this is my first CMake recipe, and indeed my first python code (except for a "hello world" some years ago ;-), I'd like some advice on a few points: - in which directory (portage group) does it belong? I added it to "win32libs-sources" but maybe "testing" or "libs" would be a better place? - the CMake file obviously only works on windows with the DirectX SDK installed, as it is currently hardcoded to use this specific configuration (I tried to stay as close to the original configure.in as possible). Do I need to make DirectX, OpenGL, etc. optional? - Also, I don't know if it works on x64 or on mingw, as I am not set up to test these. If something needs changing for these build environments, please tell me Diffs ----- /home/kde/trunk/kdesupport/emerge/portage/win32libs-sources/libsdl-src/libsdl-src-1.2.14.py PRE-CREATION /home/kde/trunk/kdesupport/emerge/portage/win32libs-sources/libsdl-src/CMakeLists.txt PRE-CREATION Diff: http://svn.reviewboard.kde.org/r/6656/diff Testing ------- I tested it on MSVC2008 and MSVC2010 and it works fine. I also included one test program from the libsdl distribution to see if opengl acceleration works Thanks, Michael
_______________________________________________ Kde-windows mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-windows
