search

calref / cboe

Classic Blades of Exile
http://spiderwebforums.ipbhost.com/index.php?/forum/12-blades-of-exile/
Other
173 stars 42 forks source link

Update to TGUI 0.9.x #288

Closed NQNStudios closed 1 year ago

NQNStudios commented 1 year ago

These are the changes I had to make to compile against TGUI 0.9.1 (the version available in the Arch Linux user repository, AUR).

The main thing is that TGUI uses its own String class now instead of SFML's.

This should fix #284 .

I had to trigger the build with scons INCLUDEPATH=$HOME/repos/cboe/deps/Catch2/include. I thought it shouldn't be necessary to pass that INCLUDEPATH manually (and as an absolute path) when Catch2 is meant to be provided as a submodule by default, but that should probably be a separate issue.

Locally, the scons build ran without error and told me All tests passed. The game launches and I've created a party and opened a scenario successfully.

CelticMinstrel commented 1 year ago

Looks good… I don't really have any reason not to merge this.