Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rudimentary support for CMake #104

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Rudimentary support for CMake #104

wants to merge 1 commit into from

Conversation

unmanned-player
Copy link

This patch adds CMake build support for various target environments and compilers. The CMake builds and compiles all demo code against SDL-1.2, SDL-2.0 X11, Wincon and DOS. It works with GNU, OpenWatcom and MSVC compilers.

FindSDL2.cmake was added as I don't know how to use find_package() in CONFIG mode. While I've heard of people using PDCurses on Apple computers, I do not own one to verify if it works.

This patch adds CMake build support for various target environments
and compilers. The CMake builds and compiles all demo code against
SDL-1.2, SDL-2.0 X11, Wincon and DOS. It works with GNU, OpenWatcom
and MSVC compilers.

FindSDL2.cmake was added as I don't know how to use  find_package()
in CONFIG mode. While I've heard of people using PDCurses on Apple
computers, I do not own one to verify if it works.
@Mizux
Copy link

Mizux commented Nov 17, 2023

You may want to use:
or CMAKE_C_COMPILER_ID MATCHES clang) to also catch Apple/ARM/Cray/XL/IBM clang based compiler.s..

ref:
https://cmake.org/cmake/help/latest/variable/CMAKE_LANG_COMPILER_ID.html
https://cmake.org/cmake/help/latest/command/if.html#matches

Repository owner deleted a comment from Bill-Gray Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants