Yes I've gotten the notifications - will take a look in a few hours. It's likely a clash between the build system I have and what's in the repo. There are some differences due to how I make the universal and redistributable packages. So I wouldn't worry about it.
On Sat, Sep 24, 2022 at 18:11, Thomas Huth <
th.huth@xxxxxxxxx> wrote:
Am Sat, 24 Sep 2022 17:48:46 +0200
schrieb Nicolas Pomarède <npomarede@xxxxxxxxxxxx>:
[...]
> > As discussed off-list with Andreas, it seems like the additional
> > target_link_libraries() statements are still necessary for the
> > sub-libraries that we build. I've added them now, together with a
> > fix on macOS that occurred when compiling without the OSX bundeling,
> > so I hope we're fine now...
>
> just to confirm that after updating with these changes, I see no problem
> when compiling for linux and also no problem when using mingw to compile
> Windows 32 bit or 64 bit versions.
Thanks for checking!
> So, all our usual target for linux/windows/macos seem to work correctly
> with the official SDL's config for cmake.
Unfortunately, it seems like it broke Troed's macOS builds:
https://sync.wtf/hatari_macos/
However, the errors in the log look very weird, like:
In file included from /Users/troed/dev/hatari/src/debug/debugui.c:16:
In file included from /opt/local/include/SDL2/SDL.h:38:
In file included from /opt/local/include/SDL2/SDL_cpuinfo.h:86:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchai
n/usr/lib/clang/12.0.0/include/immintrin.h:14:
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/clang/12.0.0/include/mmintrin.h:50:12: error: invalid conversion between vector type '__m64' (vector of 1 'long long' value) and integer type 'int' of different size
return (__m64)__builtin_ia32_vec_init_v2si(__i, 0);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
No clue what is happening there ... Troed, do you maybe have an idea?
Thomas