search

Wargus / stratagus

The Stratagus strategy game engine
GNU General Public License v2.0
622 stars 119 forks source link

War1gus fix #590

Closed Jarod42 closed 9 months ago

ipochto commented 9 months ago

I see the CI tests pass, but somehow on my PC I get this when compiling:

stratagus/src/include/video.h:44:10: fatal error: guichan/sdl/sdlimage.hpp: No such file or directory
[build]    44 | #include <guichan/sdl/sdlimage.hpp>
[build]       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~
Jarod42 commented 9 months ago

Have you updated third-party? (no need to launch update.sh ;-) ) Issue with submodules and switching branches is that it need extra flags to also apply to submodules :-/

ipochto commented 9 months ago

You're right. Updated. Now have this:

[build] stratagus/third-party/guichan/src/sdl/sdlinput.cpp:55:17: error: ‘uint32_t’ in namespace ‘std’ does not name a type; did you mean ‘wint_t’?
[build]    55 |     static std::uint32_t utf8ToUnicode(const char* text)
[build]       |                 ^~~~~~~~
[build]       |                 wint_t
[build] stratagus/third-party/guichan/src/sdl/sdlinput.cpp: In member function ‘virtual void gcn::SDLInput::pushInput(SDL_Event)’:
[build] stratagus/third-party/guichan/src/sdl/sdlinput.cpp:145:31: error: ‘utf8ToUnicode’ was not declared in this scope
[build]   145 |               keyInput.setKey(utf8ToUnicode(event.text.text));
[build]       |                               ^~~~~~~~~~~~~
Jarod42 commented 9 months ago

Shit, missing #include <cstdint> on linux ... :-/

Working on it. just include the missing header as local fix.