#2 General project maintenance & Windows build fix

Merged
38_ViTa_38 merged 7 commits from KillianPalmer/master into 38_ViTa_38/master 2 years ago

I hope the commits are fairly self-explanatory. Here I'll just explain some of the reasoning behind the changes:

  • [1273ef4] PrBoom+ compile warnings are now disabled, as these aren't really relevant to XDRE and just clutter the compilation output.
  • [8e56a55] There must exist a better fix for this issue, but this is what happens to work for me: I was getting make errors due to the rdatawad executable ("No rule to make target 'rdatawad' [...] Stop."). This was fixed by manually appending the 'exe' file extension in the relevant Makefile.am script. Now, Autotools will generate prboom-plus.wad by running rdatawad.exe as a Windows executable through Wine.
    I suspect that the correct solution would be to somehow tell Autotools to build rdatawad as a Linux executable, and then run it natively to generate prboom-plus.wad, thus not requiring Wine.
  • [d26e9bb] On my system, the general '-static' linker flag wasn't enough to statically link SDL2. The build would always end in linker errors. This was because sdl2.m4 was missing the ability to request static linker flags, which I then added myself. This flag now used in the Windows build scripts.
  • [362866a5ea] The reason the XDRE 2.21 release "didn't work" on Windows was only because of the absence of the prboom-plus.wad file. The program actually runs perfectly fine, but exits early when it can't find this file. The program even prints a warning message to the user to stdout, but because Windows GUI programs don't print stdout to the command-line like on Linux, it just looks like the program won't load.
I hope the commits are fairly self-explanatory. Here I'll just explain some of the reasoning behind the changes: * [1273ef4] PrBoom+ compile warnings are now disabled, as these aren't really relevant to XDRE and just clutter the compilation output. * [8e56a55] There must exist a better fix for this issue, but this is what happens to work for me: I was getting make errors due to the rdatawad executable ("No rule to make target 'rdatawad' [...] Stop."). This was fixed by manually appending the 'exe' file extension in the relevant Makefile.am script. Now, Autotools will generate prboom-plus.wad by running rdatawad.exe as a Windows executable through Wine. I suspect that the correct solution would be to somehow tell Autotools to build rdatawad as a Linux executable, and then run it natively to generate prboom-plus.wad, thus not requiring Wine. * [d26e9bb] On my system, the general '-static' linker flag wasn't enough to statically link SDL2. The build would always end in linker errors. This was because sdl2.m4 was missing the ability to request static linker flags, which I then added myself. This flag now used in the Windows build scripts. * [362866a5ea] The reason the XDRE 2.21 release "didn't work" on Windows was only because of the absence of the prboom-plus.wad file. The program actually runs perfectly fine, but exits early when it can't find this file. The program even prints a warning message to the user to stdout, but because Windows GUI programs don't print stdout to the command-line like on Linux, it just looks like the program won't load.
KillianPalmer commented 2 years ago
Poster

Some commits appear twice in this pull request. I have created a new one without the duplicates.

EDIT: It seems that I don't understand how git pull requests work. I didn't have to create a new request at all, it was sufficient to delete the duplicate commits in my repository.

Some commits appear twice in this pull request. I have created a new one without the duplicates. EDIT: It seems that I don't understand how git pull requests work. I didn't have to create a new request at all, it was sufficient to delete the duplicate commits in my repository.
38_ViTa_38 commented 2 years ago
Owner

Thank you.

Thank you.
This pull request has been merged successfully!
Sign in to join this conversation.
No Label
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.