search

ptitSeb / Serious-Engine

A port to the OpenPandora (and working fine on plain Linux) of the open source version of a game engine developed by Croteam for the classic Serious Sam games. Status: Working (for both FE and SE).
https://pyra-handheld.com/boards/threads/serious-sam-first-encounter.77225/
GNU General Public License v2.0
75 stars 23 forks source link

[INFO] Compilation fail for arm.v6 arm.v7 32bit platforms on Alpine LInux #28

Open bademux opened 3 years ago

bademux commented 3 years ago

Hi, It has so low priority, that i just put "info" label (maybe it is trivial to fix). Not compiling on arm v6 v7 on alpine linux (musl libc) for more info: https://github.com/bademux/serioussam-server/blob/master/Dockerfile

Thanks for the project!

#13 1318.0 /build/target/Sources/Engine/Terrain/TerrainMisc.cpp:316:17: error: 'isinff' was not declared in this scope; did you mean 'sinhf'?
#13 1318.0   316 |   #define Isinf isinff
#13 1318.0       |                 ^~~~~~
#13 1318.0 /build/target/Sources/Engine/Terrain/TerrainMisc.cpp:331:22: note: in expansion of macro 'Isinf'
#13 1318.0   331 |     rc.rc_iLeft   = (Isinf(bbox.minvect(1)))?(INDEX)0:Clamp((INDEX)(bbox.minvect(1)-0),(INDEX)0,ptrTerrain->tr_pixHeightMapWidth);
ptitSeb commented 3 years ago

I had the same issue on the Pandora. It would build with the "PANDORA" build flag I suppose (it will make a cortex-a8 arm softfp build, maybe thta needs some adaptation).

bademux commented 3 years ago

Sorry for offtopic, Are you interested in CI build for the game, github supports multiple platforms (linux, win, macos) and architectures?

ptitSeb commented 3 years ago

Why not. I have setup this kind of things for some of my other repo (like StuntCarRemake for example).

I didn't initialy did it because it was a fork of a fork.

bademux commented 3 years ago

Well, the main argument is laziness - be prepared for more bug reports. :) Let me open new issue for the discussion: https://github.com/ptitSeb/Serious-Engine/issues/29