Re: [Tuxpaint-devel] Started the port to SDL3

[Pere P. i C. <per...@gm...>]

El dg. 27 de 07 de 2025 a les 11:08 +0100, en/na Bill Kendrick va escriure:
> On Sun, Jul 27, 2025 at 11:46:04AM +0200, Pere Pujal i Carabantes wrote:
> > El dg. 27 de 07 de 2025 a les 08:23 +0900, en/na Shin-ichi TOYAMA va escriure:
> > > Hi!
> > > 
> <snip>
> > > > >   In file included from /home/kendrick/tuxpaint/sdl3-libs/SDL_mixer/src/SDL_mixer.c:24:
> > > > >   /home/kendrick/tuxpaint/sdl3-libs/SDL_mixer/src/SDL_mixer_internal.h:139:23: error: expected declaration specifiers or ‘...’ before numeric constant
> > > > >     139 |     float SDL_ALIGNED(16) position3d[4];   // we only need the X, Y, and Z coords, but the 4th element makes this SIMD-friendly.
> > > 
> > > SDL3_mixer seems to be under a very active work in progress.
> > > 
> > > I also once succeeded to build it but currently it fails with the same error bill mentioned.
> > > Then, confirmed that it could build until the commit "4e9a308a0daa63b6bd492b0ec616f86337500898".
> 
> That commit has a lot fewer source files.  MIX_Track doesn't yet
> exist; there's no "SDL_mixer_internal.h".  That said, it DID build. :-D
> 
> And Tux Paint builds (in its entirety) for me, with only this change
> to Makefile
> 
>   -SDL_LIBS+=$(shell $(PKG_CONFIG) sdl3-gfx --libs)
>   +SDL_LIBS+=-lSDL3_gfx
> 
> It knows it's built with SDL3:
> 
>   kendrick@bill-t480s:~/tuxpaint/perepujal-tuxpaint$ tuxpaint --verbose-version 
>   
>   Tux Paint
>     Version 0.9.36 (2025-07-27)
>   
>   Built with these options:
>     SDL version 3.2.18
> 
> However, it segfaults after the splash screen appears.
> (It was built WITH all magic tools, this time around.)
> 
> I'll keep investigating when I can :)
> 
> 
> > > I will wait for a while until it would be more stable.
> 
> We might need to, hehe.

I agree too.


> > 
> > Actually, with commit 59a2f77645404aee3ab541c6c860970da7572bed from July 26
> > builds fine for me.

The SDL3_mixer library builds fine, but after installed it,   
breaks the Tux Paint build, so Tux Paint would need to be adapted   
to the changes in the mixer lib.

For now, I am going to downgrade the mixer in my box to the  
4e9a308a0daa63b6bd492b0ec616f86337500898 commit until more people   
is able to compile the main branch of SDL3_Mixer.


> > 
> > Starting at the root of the source, rm the "build" directory if it has been created before, then:
> > mkdir build && cd build && cmake -S .. -B . && make
> 
> That commit continued to not build, due to `position3d` and `position`
> differences in the struct described in "SDL_mixer_internal.h", and how
> it was used elsewhere in the SDL_mixer sources.  (Even after a `git pull`
> on "master" branch.)


Strange, could be that a different compiler/make/whatever would not throw errors?  
It builds here with no complaints.  
In case of that helps:

gcc --version  
gcc (Debian 14.2.0-19) 14.2.0

make --version  
GNU Make 4.4.1

ldd --version  
ldd (Debian GLIBC 2.41-6) 2.41

cmake --version  
cmake version 3.18.4


HTH  
Pere