-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add CMake build system and move/rename examples #45
Conversation
@jopadan I used to build adikted by typing 'make all' in WSL in adikted/mapslang. Trying to build your PR and it does not find a makefile. How can one build now? |
Good work, just fix the |
It would be nice if we could retain the possibility of building |
CMakeLists.txt
Outdated
add_test( puttrain puttrain ) | ||
|
||
add_executable( viewmap ${EXAMPLES_VIEWMAP_SOURCES} ) | ||
target_link_libraries( viewmap PUBLIC adikted m dl ${SDL_LIBRARIES}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change ${SDL_LIBRARIES}
to ${SDL_LIBRARY}
as it only exists in CMake 3.19 and onwards
CMakeLists.txt
Outdated
add_test( viewmap viewmap ) | ||
|
||
add_executable( putemple ${EXAMPLES_PUTEMPLE_SOURCES} ) | ||
target_link_libraries( putemple PRIVATE adikted m dl ${SDL_LIBRARIES}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change ${SDL_LIBRARIES}
to ${SDL_LIBRARY}
as it only exists in CMake 3.19 and onwards
CMakeLists.txt
Outdated
file( GLOB_RECURSE EXAMPLES_PUTEMPLE_SOURCES examples/putemple/*.c | ||
examples/putemple/*.h ) | ||
|
||
find_package( SDL REQUIRED SDL::Main ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider making it optional so if SDL is not found the viewmap
and putemple
example projects are not built. Not super important though.
Continued in #48 |
No description provided.