Add support for building with CMake #327
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
skalleAnka
force-pushed
the
cmake-support
branch
2 times, most recently
from
a04f471 to
08ce92c
Compare
|
Now also tested with MINGW - had some trouble with the SDL2main library but it should be sorted now. |
skalleAnka
force-pushed
the
cmake-support
branch
2 times, most recently
from
c0160f3 to
0758da2
Compare
|
Modified it to extract version number from quakedef.h, making it even more maintenance free. |
skalleAnka
force-pushed
the
cmake-support
branch
from
0758da2 to
08a0bb4
Compare
|
Updated PR to remove CD support as in master |
skalleAnka
force-pushed
the
cmake-support
branch
2 times, most recently
from
82f5b87 to
706207a
Compare
skalleAnka
force-pushed
the
cmake-support
branch
from
706207a to
b10d27a
Compare
|
Added in a8de85b, thank you! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This will make it possible to use CMake for building.
On Linux most IDEs/editors CMake seems to be the default option these days. On Windows it is the best option when using VSCode and is also supported natively by Visual Studio as an alternative to project files.
On linux this will find SDL, CURL and the codec packages automatically if they are installed. On Windows it uses the bundled libraries. I think it should be possible to reconfigure on Windows to use packages from some other location but I have not tested that.
It should be largely self-maintaining: If new source files are added to (or removed from) the Quake folder, it will pick them up without modification of the scripts.
I have tested this with Debian 12 Linux (GCC and Clang) and with Visual Studio 2022 and it seems to work.