Skip to content

Commit 0476871

Browse files
author
Swagtoy
committed
Remove supertux2_lib and supertux2_c
1 parent c72db8b commit 0476871

File tree

2 files changed

+44
-45
lines changed

2 files changed

+44
-45
lines changed

CMakeLists.txt

+41-42
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,6 @@ endif()
142142
#include(SuperTux/ProvideTinygettext)
143143
#include(SuperTux/ProvideSDL2_ttf) # add back?
144144

145-
146145
# Find dependencies
147146
find_package(SDL2 CONFIG REQUIRED)
148147
#find_package(PNG CONFIG REQUIRED)
@@ -162,6 +161,7 @@ include(SuperTux/ProvideDiscord)
162161
include(SuperTux/ProvideSexpcpp)
163162
include(SuperTux/ProvidePartioZip)
164163
include(SuperTux/ProvideOpenGL)
164+
include(SuperTux/ProvideTinygettext)
165165

166166
## Build stuff
167167
include(SuperTux/BuildVersion)
@@ -187,76 +187,74 @@ list(SORT SUPERTUX_RESOURCES)
187187
include(SuperTux/CompileAmalgation)
188188

189189
## Generate supertux executable in the right place
190-
190+
# Is this needed? - Swagtoy
191191
#set(EXECUTABLE_OUTPUT_PATH ${CMAKE_CURRENT_SOURCE_DIR})
192-
set(EXECUTABLE_OUTPUT_PATH ${CMAKE_BINARY_DIR})
192+
#set(EXECUTABLE_OUTPUT_PATH ${CMAKE_BINARY_DIR})
193193

194194
## Add target for supertux binary
195195

196-
add_library(supertux2_c OBJECT ${SUPERTUX_SOURCES_C})
197-
add_library(supertux2_lib STATIC ${CMAKE_BINARY_DIR}/version.h ${SUPERTUX_SOURCES_CXX} ${SUPERTUX_RESOURCES} $<TARGET_OBJECTS:supertux2_c>)
198-
target_include_directories(supertux2_lib PUBLIC ${CMAKE_BINARY_DIR} src/)
199-
200-
if(WIN32)
201-
add_executable(supertux2 WIN32 src/main.cpp ${CMAKE_CURRENT_SOURCE_DIR}/data/images/engine/icons/supertux.rc)
202-
target_link_libraries(supertux2 LibSDL2main)
203-
else()
204-
add_executable(supertux2 src/main.cpp)
205-
endif()
206-
target_link_libraries(supertux2 supertux2_lib)
196+
add_executable(supertux2 WIN32 ${CMAKE_BINARY_DIR}/version.h ${SUPERTUX_SOURCES_C} ${SUPERTUX_SOURCES_CXX} ${SUPERTUX_RESOURCES} src/main.cpp)
197+
# TODO Is this even needed?
198+
# $<$<BOOL:WIN32>:{CMAKE_CURRENT_SOURCE_DIR}/data/images/engine/icons/supertux.rc>
199+
target_include_directories(supertux2 PUBLIC ${CMAKE_BINARY_DIR} src/)
207200

208-
set_target_properties(supertux2_lib PROPERTIES OUTPUT_NAME supertux2_lib)
209-
set_target_properties(supertux2_lib PROPERTIES COMPILE_FLAGS "${SUPERTUX2_EXTRA_WARNING_FLAGS}")
201+
set_target_properties(supertux2 PROPERTIES OUTPUT_NAME "supertux2")
202+
set_target_properties(supertux2 PROPERTIES COMPILE_FLAGS "${SUPERTUX2_EXTRA_WARNING_FLAGS}")
210203

211-
if(WIN32 AND NOT VCPKG_BUILD)
212-
if(NOT MINGW)
204+
#if(WIN32 AND NOT VCPKG_BUILD)
205+
#if(NOT MINGW)
213206
## Copy dlls on windows
214-
add_custom_command(TARGET supertux2_lib POST_BUILD
215-
COMMAND ${CMAKE_COMMAND} -E copy_directory
216-
"${DEPENDENCY_FOLDER}/dll"
217-
$<TARGET_FILE_DIR:supertux2_lib>)
218-
endif()
219-
endif()
207+
# add_custom_command(TARGET supertux2_lib POST_BUILD
208+
# COMMAND ${CMAKE_COMMAND} -E copy_directory
209+
# "${DEPENDENCY_FOLDER}/dll"
210+
# $<TARGET_FILE_DIR:supertux2_lib>)
211+
#endif()
212+
#endif()
220213

221214
## Some additional include paths
222-
target_include_directories(supertux2_lib SYSTEM PUBLIC
215+
target_include_directories(supertux2 SYSTEM PUBLIC
223216
external/findlocale/
224217
external/obstack/
225218
)
226219

227220
# Include altivec wrapper on ppc
228221
if(CMAKE_SYSTEM_PROCESSOR MATCHES "^ppc.*")
229-
target_include_directories(supertux2_lib PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/src/ppc)
222+
target_include_directories(supertux2 PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/src/ppc)
230223
endif()
231224

232225
## Link supertux binary with squirrel and other libraries
233226

234-
target_link_libraries(supertux2_lib PUBLIC LibSimpleSquirrel)
235-
target_link_libraries(supertux2_lib PUBLIC LibTinygettext)
236-
target_link_libraries(supertux2_lib PUBLIC LibSexp)
237-
target_link_libraries(supertux2_lib PUBLIC LibSavePNG)
238-
target_link_libraries(supertux2_lib PUBLIC LibPartioZip)
239-
target_link_libraries(supertux2_lib PUBLIC OpenAL::OpenAL)
240-
target_link_libraries(supertux2_lib PUBLIC glm::glm)
241-
target_link_libraries(supertux2_lib PUBLIC fmt::fmt)
242-
target_link_libraries(supertux2_lib PUBLIC $<IF:$<TARGET_EXISTS:PhysFS::PhysFS>,PhysFS::PhysFS,PhysFS::PhysFS-static>)
227+
target_link_libraries(supertux2 PUBLIC LibSimpleSquirrel)
228+
target_link_libraries(supertux2 PUBLIC LibTinygettext)
229+
target_link_libraries(supertux2 PUBLIC LibSexp)
230+
target_link_libraries(supertux2 PUBLIC LibSavePNG)
231+
target_link_libraries(supertux2 PUBLIC LibPartioZip)
232+
target_link_libraries(supertux2 PUBLIC OpenAL::OpenAL)
233+
target_link_libraries(supertux2 PUBLIC glm::glm)
234+
target_compile_definitions(supertux2 PUBLIC GLM_ENABLE_EXPERIMENTAL)
235+
target_link_libraries(supertux2 PUBLIC fmt::fmt)
236+
target_link_libraries(supertux2 PUBLIC $<IF:$<TARGET_EXISTS:PhysFS::PhysFS>,PhysFS::PhysFS,PhysFS::PhysFS-static>)
243237

244238
if(NOT EMSCRIPTEN)
245-
target_link_libraries(supertux2_lib PUBLIC $<IF:$<TARGET_EXISTS:SDL2_ttf::SDL2_ttf>,SDL2_ttf::SDL2_ttf,SDL2_ttf::SDL2_ttf-static>)
246-
target_link_libraries(supertux2_lib PUBLIC
239+
target_link_libraries(supertux2 PUBLIC
240+
# SDL2_image
247241
$<IF:$<TARGET_EXISTS:SDL2_image::SDL2_image>,SDL2_image::SDL2_image,SDL2_image::SDL2_image-static>
242+
# SDL2_ttf
243+
$<IF:$<TARGET_EXISTS:SDL2_ttf::SDL2_ttf>,SDL2_ttf::SDL2_ttf,SDL2_ttf::SDL2_ttf-static>
244+
# SDL2 main (windows?)
248245
$<TARGET_NAME_IF_EXISTS:SDL2::SDL2main>
246+
# SDL2
249247
$<IF:$<TARGET_EXISTS:SDL2::SDL2>,SDL2::SDL2,SDL2::SDL2-static>)
250-
target_link_libraries(supertux2_lib PUBLIC Ogg:ogg Vorbis::vorbis Vorbis::vorbisfile)
251-
target_link_libraries(supertux2_lib PUBLIC CURL::libcurl)
248+
target_link_libraries(supertux2 PUBLIC Ogg::ogg Vorbis::vorbis Vorbis::vorbisfile)
249+
target_link_libraries(supertux2 PUBLIC CURL::libcurl)
252250
endif()
253251

254252
if(HAVE_OPENGL)
255-
target_link_libraries(supertux2_lib PUBLIC LibOpenGL)
253+
target_link_libraries(supertux2 PUBLIC LibOpenGL)
256254
endif()
257255

258256
if(ENABLE_DISCORD)
259-
target_link_libraries(supertux2_lib PUBLIC LibDiscord)
257+
target_link_libraries(supertux2 PUBLIC LibDiscord)
260258
endif()
261259

262260
## Install stuff
@@ -266,7 +264,8 @@ include(SuperTux/BuildInstall)
266264
configure_file(config.h.cmake ${CMAKE_BINARY_DIR}/config.h )
267265

268266
## Build tests
269-
include(SuperTux/BuildTests)
267+
# TODO Move to CMake Tests
268+
#include(SuperTux/BuildTests)
270269

271270
## CPack/Installation-specific stuff
272271
include(SuperTux/BuildCPack)

mk/cmake/SuperTux/BuildInstall.cmake

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
if(WIN32 AND NOT UNIX)
2-
install(DIRECTORY $<TARGET_FILE_DIR:supertux2_lib>/
3-
DESTINATION ${INSTALL_SUBDIR_BIN}
4-
FILES_MATCHING PATTERN "*.dll")
2+
#install(DIRECTORY $<TARGET_FILE_DIR:supertux2>/
3+
# DESTINATION ${INSTALL_SUBDIR_BIN}
4+
# FILES_MATCHING PATTERN "*.dll")
55

66
install(FILES ${CMAKE_CURRENT_SOURCE_DIR}/data/images/engine/icons/supertux.png
77
${CMAKE_CURRENT_SOURCE_DIR}/data/images/engine/icons/supertux.ico

0 commit comments

Comments
 (0)