On Fri, Mar 26, 2021 at 05:20:06PM +0100, Charlene Wendling wrote:
> Hi !

> > http://build-failures.rhaalovely.net/sparc64/2021-03-21/games/egoboo.log
> > http://build-failures.rhaalovely.net/powerpc/2021-02-22/games/egoboo.log
> > http://build-failures.rhaalovely.net/mips64/2021-03-21/games/egoboo.log

> There is a typo in the big endian code there. It's 'SDL_Swap32'.

> With the below diff, it builds and works fine on macppc, colors are
> not off. I added a safety revision bump as well.

> Comments/feedback are welcome,

> Charlène.

Well, it fixed the build of egoboo in my current sparc64 build.

ok kmos

> 
> [0] https://wiki.libsdl.org/SDL_Swap32
> 
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/games/egoboo/Makefile,v
> retrieving revision 1.11
> diff -u -p -u -p -r1.11 Makefile
> --- Makefile  8 Mar 2021 18:20:14 -0000       1.11
> +++ Makefile  26 Mar 2021 16:03:56 -0000
> @@ -3,6 +3,7 @@
>  COMMENT =            dungeon-crawling action-RPG adventure
>  
>  DISTNAME =           egoboo-2.8.1
> +REVISION =           0
>  CATEGORIES =         games
>  
>  HOMEPAGE =           http://egoboo.sourceforge.net
> @@ -21,7 +22,8 @@ LIB_DEPENDS =               devel/sdl-image \
>  FIX_CRLF_FILES =     src/Makefile src/enet/Makefile src/game/Makefile \
>                       src/game/extensions/ogl_texture.c \
>                       src/game/game.c \
> -                     src/game/platform/file_linux.c
> +                     src/game/platform/file_linux.c \
> +                     src/game/md2.c
>  
>  USE_GMAKE =          Yes
>  WRKSRC =             ${WRKDIST}/src
> Index: patches/patch-src_game_md2_c
> ===================================================================
> RCS file: patches/patch-src_game_md2_c
> diff -N patches/patch-src_game_md2_c
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-src_game_md2_c      26 Mar 2021 16:03:56 -0000
> @@ -0,0 +1,16 @@
> +$OpenBSD$
> +
> +Fix the build on BE_ARCHS.
> +
> +Index: src/game/md2.c
> +--- src/game/md2.c.orig
> ++++ src/game/md2.c
> +@@ -511,7 +511,7 @@ MD2_Model_t* md2_load( const char * szFilename, MD2_Mo
> +                 int i;
> +                 for ( i = 0; i < cmd->command_count; i++ )
> +                 {
> +-                    cmd->data[i].index = SDL_swap32( cmd->data[i].s );
> ++                    cmd->data[i].index = SDL_Swap32( cmd->data[i].s );
> +                     cmd->data[i].s     = ENDIAN_FLOAT( cmd->data[i].s );
> +                     cmd->data[i].t     = ENDIAN_FLOAT( cmd->data[i].t );
> +                 };
> 

Reply via email to