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

[Bill K. <nb...@so...>]

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.


> > Just FYI.
> 
> Actually, with commit 59a2f77645404aee3ab541c6c860970da7572bed from July 26
> builds fine for me.
> 
> 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!

-bill!