[Scummvm-devel] LUA on WinCE and PS2

[Max Horn]

The port is not to blame when lua is using non-portable APIs, as has been mentioned here on several occasions. So, to solve this, don't add compatibility layers to the ports (eg a fake errno.h), the sword25 code should be made portable.

Max

Am 14.05.2011 um 11:43 schrieb Filippos Karapetis <bluegr at gmail.com>:

> Hello
> 
> I've recently dropped the libpng requirement for the sword25 engine. This allowed the engine to be compiled
> for the majority of platforms we support (with a few patches here and there). However, the engine (and in
> particular, the LUA library it uses) fails to compile under WinCE and PS2, and I would like some opinions
> on how to handle them.
> 
> PS2: The errors there seem... odd. Seems like there's some weird problem with libc. Can the PS2 porter
> supply more information on them?
> /opt/toolchains/ps2/tools/ee/lib/gcc-lib/ee/3.2.2/../../../../ee/lib/libc.a(unlinkr.o)(.text+0x1c): In function `_unlink_r':
> ../../../../../newlib/libc/reent/unlinkr.c:54: undefined reference to `unlink'
> /opt/toolchains/ps2/tools/ee/lib/gcc-lib/ee/3.2.2/../../../../ee/lib/libc.a(linkr.o)(.text+0x24): In function `_link_r':
> ../../../../../newlib/libc/reent/linkr.c:61: undefined reference to `link'
> 
> Also, there are tons of warnings like these:
> In file included from ../../src-master/src/engines/sword25/sword25.cpp:46:
> ../../src-master/src/engines/sword25/kernel/kernel.h:73: warning: `class 
>    Sword25::Kernel' only defines a private destructor and has no friends
> 
> There's a comment inside that class that the constructor and destructor are private to make the class
> a singleton... this sounds odd to me. Why create a singleton in this manner inside engine code in the first place?
> 
> Now, concerning WinCE... AFAIK, this port misses a ton of headers from the standard C library.
> The current error thrown is that errno.h is missing. According to a post I found while Googling, we could use a
> minimal errno.h:
> http://mailman.videolan.org/pipermail/vlc-devel/2009-November/068586.html
> and the minimal errno.h can be found at:
> http://altair.videolan.org/~linkfanel/errno
> 
> Is the aforementioned solution viable? What else is missing from WinCE?
> 
> Regards
> Filippos
> 
> -- 
> "Experience is the name every one gives to their mistakes" - Oscar Wilde 
> ------------------------------------------------------------------------------
> Achieve unprecedented app performance and reliability
> What every C/C++ and Fortran developer should know.
> Learn how Intel has extended the reach of its next-generation tools
> to help boost performance applications - inlcuding clusters.
> http://p.sf.net/sfu/intel-dev2devmay
> _______________________________________________
> Scummvm-devel mailing list
> Scummvm-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/scummvm-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scummvm.org/pipermail/scummvm-devel/attachments/20110514/4c7460de/attachment.html>