Review Request: Add libsdl-1.2 to the windows portage repository for emerge

[Michael Drüing]

-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-windows/attachments/20110402/2ea73284/attachment.htm