[09:38:05] Dominus: Marzo, I didn't notice that the ci-macos does not test the gimp plugin and I haven't build it for a long time. Now I tried and ran into build errors https://pastebin.com/ZcvPP5e5
[09:39:58] Marzo: It is not building as c++14 for whatever reason
[09:54:04] Dominus: huh... if I force c++14 by adding it to the c/cpp/cf/cxxflags ( --std=c++14) it builds but doesn't link with multiple errors https://pastebin.com/49Pk1xA5
[10:01:07] Marzo: You probably also need to add c++14 flags for linker
[10:04:01] Dominus: it's also set for LDFLAGS
[10:04:23] Marzo: Oh, right, I see it
[10:04:25] Marzo: Hm
[10:05:59] Marzo: Try adding a -stdlib=libc++
[10:06:28] Marzo: Hm, wait
[10:06:39] Dominus: ok :)
[10:07:59] Marzo: It is not linking several Exult things
[10:53:35] Dominus: I really don't understand what's happening here. I tried several things now but always end up with this
[10:58:33] azeem: I get the following warnings on autoreconf (thought they've been around a long time I guess): http://paste.debian.net/1237452/
[11:00:52] Dominus: yes, I noticed, too that the latest autoreconf is no longer happy with our configure.ac
[11:02:23] azeem: I also get linking errors even though configure went through fine: /usr/bin/ld: /tmp/ccA2D2hK.o: undefined reference to symbol '_ZNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEE9_M_appendEPKcm'
[11:05:32] Dominus: linking which part or everywhere?
[11:06:20] azeem: I think in mapedit, u7shp, but others as well
[11:06:35] azeem: so both libgtk3.0 and libgtk2.0 get dragged in
[11:06:41] azeem: maybe a Debian issue then
[11:07:17] azeem: ah no, github-action CI also installs both
[11:07:22] azeem: let me compare build deps
[11:09:09] azeem: /usr/bin/ld: /usr/lib/x86_64-linux-gnu/libexiv2.so.27: error adding symbols: DSO missing from command line
[11:09:20] azeem: that's the second error line, which seems important as well :)
[11:11:11] Dominus: bah, I hate missing symbols
[11:14:46] azeem: exiv seems to come in via libgimp-dev
[11:14:55] azeem: it's not mention in libgimp's .oc
[11:14:59] azeem: eh, .pc
[11:18:43] azeem: when you make clean, reconfigure and jump into mapedit to make, you'll eventually get a "no rule to make conf/something:", I wonder whether that could be more friendly, i.e. build the dependencies first
[11:27:37] azeem: I've added all the deps from github-action's linux CI as well as the configure opts, and the undefined reference persists
[11:28:25] azeem: anybody know what "ubuntu-latest" is exactly?
[11:29:05] azeem: probably 20.04
[11:34:18] Dominus: yes https://github.com/actions/virtual-environments
[11:35:31] azeem: I get the same issue on Debian stable though
[11:43:28] azeem: what's sticks out if I compare the command-lines between gh-actions and debian build is "-Wl,-z,relro -Wl,--as-needed" so I'm trying without that now
[11:46:27] azeem: no that's not it either
[11:46:56] azeem: those gazillion of GTK/ATK warnings don't help comparing things
[11:47:32] azeem: well, glib/atk I guess
[12:05:40] azeem: also I wonder why u7shp is being linked by gcc and not g++?
[12:16:35] Dominus: maybe that's why it won't build on macOS?
[12:18:33] Dominus: Marzo, it seems the gimp plugin never built on macOS since you made it a c++ program. first the makefile.am didn't allow it to find the exult includes and then it fails with either c++14 not being defined or if that is with linking failing
[12:19:08] Marzo: Or in linux, as I am investigating it under WSL
[12:19:23] Marzo: It seems it was only building on Windows because it does not use gimptool
[12:31:01] azeem: well the linux-ci gh action seems to build fine right?
[13:16:21] azeem: I tried with a focal build chroot and I indeed don't see the exiv2 linking error, however, I get a similar error with libstdc++ there
[13:16:34] azeem: /usr/bin/ld: /tmp/ccoNBIq1.o: undefined reference to symbol '_ZNSt6localeD1Ev@@GLIBCXX_3.4'
[13:16:37] azeem: /usr/bin/ld: /usr/lib/x86_64-linux-gnu/libstdc++.so.6: error adding symbols: DSO missing from command line
[13:28:02] azeem: oh
[13:29:49] azeem: I have the feeling the build system just doesn't propagate errors correctly, or is github-action supposed to be green in such a case?
[13:35:01] Dominus: I *think* because it is using gimptool the errors don't stop it
[13:35:25] Marzo: I just pushed a fix
[13:36:17] azeem: Dominus: yeah exactly, looks like a bug in gimptool
[13:37:31] azeem: Marzo: thanks, will try
[13:37:49] Marzo: The fix is to avoid using gimptool for building
[13:38:04] Marzo: And just use it to get include and link flags
[13:45:04] azeem: /usr/share/games/exult/bg_{paperdol,mr_faces}.vga are no longer around, is that expected?
[13:47:36] azeem: anyway, it builds fine
[13:47:56] azeem: now I need to figure out which new data files belong to exult, and which to exult-studio
[13:48:00] azeem: but I'll do that later
[13:49:09] Marzo: Dominus: with 0356e19f, build now fails for me :/
[13:50:27] Marzo: Command line shows a -isystem -isystem path: at the end
[13:51:06] Dominus: come on
[13:51:45] Marzo: Which is what the -I was there for in the first place
[13:51:48] azeem: installation of x-shapefile.xml does not seem to honor DESTDIR
[13:51:50] Dominus: on the up side, u7shp now builds for me as well :)
[13:52:16] Marzo: What is the output of "pkg-config --cflags gimpui-2.0 gtk+-2.0" on that system you had that reproduced the issue?
[13:52:22] Dominus: azeem: data/estudio/*, data/exult_studio.glade but it also needs exult.cfg to find the normal exult.flx files
[13:52:38] Dominus: one moment, starting my vm
[13:53:15] azeem: what is u7voice2syx for?
[13:53:49] azeem: I guess it should go into the exult package, or is it for game development?
[13:54:06] Dominus: Marzo: https://pastebin.com/UjSsfnBU
[13:54:26] Dominus: azeem, no idea if it has any use these days
[13:54:26] Marzo: @azeem: It should probably not be distributed at all
[13:54:41] azeem: ok
[13:54:59] azeem: it turned up cause I added some additional build-deps or configure options
[13:57:31] Marzo: Dominus: I found the real issue
[13:58:32] Marzo: For some reason, pkg-config is giving out 'mixed' Windows paths on the systems where having the -I breaks the build, while it is giving msys2 paths on mine
[13:59:13] Marzo: Can you also post the result of "echo $MSYSTEM_PREFIX" on that system?
[14:00:29] Dominus: it's /mingw64
[14:01:06] Marzo: There is why the subst was not working
[14:01:23] Marzo: Now why is pkg-config behaving differently
[14:02:14] Dominus: can you get the msys mingw version somehow?
[14:06:20] Marzo: Can you look for the following files and post their contents: /mingw32/lib/pkgconfig/gtk+-2.0.pc /mingw64/lib/pkgconfig/gtk+-2.0.pc /usr/lib/pkgconfig/gtk+-2.0.pc
[14:21:12] Marzo: Dominus: or better yet, can you post the result of "which pkg-config"?
[14:42:49] Dominus: "/mingw64/bin/pkg-config"
[14:43:58] Dominus: and https://pastebin.com/rUNVaPE3
[14:45:11] Dominus: there is no gtk+-2.0.pc file in the other two locations only in mingw64/lib/pkgconfig
[14:45:26] Marzo: I figured that last one
[14:45:36] Marzo: And for what is worth, here is the real issue: https://github.com/msys2/MINGW-packages/issues/5650
[14:46:11] Marzo: Basically, there can be up to 3 different versions of pkg-config.exe in a msys2 install: a 32-bit one, a 64-bit one, and the msys one
[14:46:25] Marzo: The msys one was in my path and returns Unix-style paths
[14:46:40] Marzo: The other two return Windows paths with backslashes
[14:46:59] Marzo: And the makefile was assuming Unix style
[14:48:20] Dominus: so an older install that had a lot going on over the months might be using the msys one, while a fresh install (as on the ci and mine (sort of)) use the mingw64 one...
[14:49:03] Marzo: And the "fix" actually broke the intent of the original fix, but just happened to work because it had the effect of removing the bad path from includes
[14:49:16] Marzo: So I will revert the change and apply a proper fix
[14:49:22] Dominus: thanks!
[14:51:11] Dominus: marzo: as for having these fixes in the v1.8.x, should I cerry pick (which I know how to do) or merge it somehow (which I don't know, especiallyas I reverted the exult as lib commit)
[14:51:14] Dominus: ?
[14:51:58] Marzo: Cherry-pick would be better
[15:28:43] Marzo: Dominus: fixed
[17:31:48] Dominus: thank you Marzo
[18:08:43] Dominus: for "fun" I tried submitting the iOS version to the Apple AppStore... First review didn't like the name (Exult-iOS because Exult was already taken) AND doesn't like that it shows an error on first launch because of the missing game files :)
[18:09:04] Dominus: This might stop Exult entirely on the App Store...
[18:15:02] Philantrop: Dominus: Exult's GPLv2 licence would prevent it from being published on the Appstore, wouldn't it?
[18:17:08] Dominus: no, it doesn't prevent it from being published but anyone that wants to exercise his rights as contributor to Exult can have it pulled
[18:21:11] Philantrop: Ah, ok, I thought I dimly remembered some clause in the Appstore's terms and conditions prohibiting GPL-licenced software.
[18:27:43] Dominus: I half expect someone to file a claim and have it pulled eventually - basically *I* am breaking the GPL, not Apple.