Commit 3ea11f9acbb876ab6b3261cff458fe69f4b96934
2021-04-18T06:23:20
Make SDL2Config.cmake export SDL2_LIBRARIES etc (on Windows) still WIP and other platforms than windows still TODO and open questions etc see https://github.com/libsdl-org/SDL/issues/4004
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115
diff --git a/SDL2Config.cmake b/SDL2Config.cmake
index 4a5f646..fd22ffd 100644
--- a/SDL2Config.cmake
+++ b/SDL2Config.cmake
@@ -1 +1,96 @@
include("${CMAKE_CURRENT_LIST_DIR}/SDL2Targets.cmake")
+
+
+# cuts off the path at the last (back)slash
+function(my_dirname FILEPATH RETVAL)
+ string(FIND ${FILEPATH} "/" slashPos REVERSE)
+ string(FIND ${FILEPATH} "\\" bsPos REVERSE) # TODO: untested, cmake always gave me forward slashes
+ if(DEFINED slashPos)
+ if( (DEFINED bsPos) AND (${bsPos} GREATER ${slashPos}) )
+ set(slashPos, ${bsPos})
+ endif()
+ elseif(DEFINED bsPos)
+ set(slashPos, ${bsPos})
+ endif()
+ if(DEFINED slashPos)
+ string(SUBSTRING ${FILEPATH} 0 ${slashPos} dirname)
+ set(${RETVAL} ${dirname} PARENT_SCOPE)
+ endif()
+endfunction()
+
+
+if(WIN32)
+ # this seems to work for both MSVC and MINGW+MSYS and with both SDL2Config/Target.cmake from vcpkg
+ # and from building myself with cmake from latest git
+
+ # ok, the headers are easy - but note that this adds both .../include and .../include/SDL2/
+ # while the SDL2_INCLUDE_DIRS of sdl2-config.cmake only add ...include/SDL2/
+ # But at least if building worked with sdl2-config.cmake it will also work with this.
+ get_target_property(SDL2_INCLUDE_DIRS SDL2::SDL2 INTERFACE_INCLUDE_DIRECTORIES)
+
+ # get the paths to the .lib files for both SDL2 and SDL2main
+ # they could be in lots of different properties..
+ get_target_property(sdl2implib SDL2::SDL2 IMPORTED_IMPLIB_RELEASE)
+ get_target_property(sdl2implibdbg SDL2::SDL2 IMPORTED_IMPLIB_DEBUG)
+
+ # TODO: for all this bla_RELEASE bla_DEBUG etc stuff, maybe only try the ones
+ # from get_target_property(available_configs SDL2::SDL2 IMPORTED_CONFIGURATIONS) ?
+ # OR otherwise at least try all of NOCONFIG, RELEASE, MINSIZEREL, RELWITHDEBINFO, DEBUG (possibly in that order)?
+
+ get_target_property(sdl2mainimplib SDL2::SDL2main IMPORTED_IMPLIB_RELEASE)
+ if(NOT sdl2mainimplib)
+ get_target_property(sdl2mainimplib SDL2::SDL2main IMPORTED_LOCATION_RELEASE)
+ endif()
+ get_target_property(sdl2mainimplibdbg SDL2::SDL2main IMPORTED_IMPLIB_DEBUG)
+ if(NOT sdl2mainimplibdbg)
+ get_target_property(sdl2mainimplibdbg SDL2::SDL2main IMPORTED_LOCATION_DEBUG)
+ endif()
+
+ if( sdl2implib AND sdl2mainimplib AND sdl2implibdbg AND sdl2mainimplibdbg )
+ # we have both release and debug builds of SDL2 and SDL2main, so use this ugly
+ # generator expression in SDL2_LIBRARIES to support both in MSVC, depending on build type configured there
+ set(SDL2_LIBRARIES $<IF:$<CONFIG:Debug>,${sdl2mainimplibdbg},${sdl2mainimplib}> $<IF:$<CONFIG:Debug>,${sdl2implibdbg},${sdl2implib}>)
+ else()
+ if(NOT sdl2implib) # try to get it from other properties
+ foreach(prop IMPORTED_IMPLIB IMPORTED_IMPLIB_NOCONFIG IMPORTED_IMPLIB_DEBUG IMPORTED_LOCATION_RELEASE IMPORTED_LOCATION_DEBUG)
+ get_target_property(sdl2implib SDL2::SDL2 ${prop})
+ if(sdl2implib)
+ message(STATUS "succeeded with ${prop} => ${sdl2implib}")
+ break()
+ endif()
+ message(STATUS "no luck with ${prop}")
+ endforeach()
+ endif()
+ if(NOT sdl2mainimplib)
+ foreach(prop IMPORTED_IMPLIB IMPORTED_IMPLIB_NOCONFIG IMPORTED_IMPLIB_DEBUG IMPORTED_LOCATION_RELEASE IMPORTED_LOCATION_DEBUG)
+ get_target_property(sdl2mainimplib SDL2::SDL2main ${prop})
+ if(sdl2mainimplib)
+ message(STATUS "succeeded with ${prop} => ${sdl2mainimplib}")
+ break()
+ endif()
+ message(STATUS "no luck with ${prop}")
+ endforeach()
+ endif()
+
+ if( sdl2implib AND sdl2mainimplib )
+ set(SDL2_LIBRARIES ${sdl2mainimplib} ${sdl2implib})
+ else()
+ message(FATAL_ERROR, "SDL2::SDL2 and/or SDL2::SDL2main don't seem to contain any kind of IMPORTED_IMPLIB")
+ endif()
+ endif()
+
+ # NOTE: SDL2_LIBRARIES now looks like "c:/path/to/SDL2main.lib;c:/path/to/SDL2.lib"
+ # which is different to what it looks like when coming from sdl2-config.cmake
+ # (there it's more like "-L${SDL2_LIBDIR} -lSDL2main -lSDL2" - and also -lmingw32 and -mwindows)
+ # This seems to work with both MSVC and MinGW though, while the other only worked with MinGW
+
+ my_dirname(${sdl2implib} SDL2_LIBDIR)
+
+ my_dirname(${SDL2_LIBDIR} SDL2_EXEC_PREFIX) # the exec prefix is one level up from lib/ - TODO: really, always? at least on Linux there's /usr/lib/x86_64-bla-blub/libSDL2-asdf.so.0 ..
+ set(SDL2_PREFIX ${SDL2_PREFIX}) # TODO: could this be somewhere else? parent dir of include or sth?
+
+elseif() # not windows
+ # TODO: about the same but don't have to look at *_IMPLIB* as that's windows specific, it should be in IMPORTED_LOCATION*
+ # TODO: make SDL2_LIBRARIES look more like the original (instead of path to libSDL2.so)?
+ # TODO: anything special about macOS? I can't test that, I have no Mac
+endif()
diff --git a/sdl2-config.cmake.in b/sdl2-config.cmake.in
index e528a88..a6fa051 100644
--- a/sdl2-config.cmake.in
+++ b/sdl2-config.cmake.in
@@ -23,7 +23,8 @@ if(NOT TARGET SDL2::SDL2)
INTERFACE_INCLUDE_DIRECTORIES "@includedir@/SDL2"
IMPORTED_LINK_INTERFACE_LANGUAGES "C"
IMPORTED_LOCATION "@libdir@/${CMAKE_SHARED_LIBRARY_PREFIX}SDL2${CMAKE_SHARED_LIBRARY_SUFFIX}"
- INTERFACE_LINK_LIBRARIES "${SDL2_EXTRA_LINK_FLAGS}")
+ # TODO: on windows set IMPORTED_IMPLIB to SDL2.lib
+ INTERFACE_LINK_LIBRARIES "${SDL2_EXTRA_LINK_FLAGS}") # TODO: really? SDL2Config.cmake doesn't do this
add_library(SDL2::SDL2-static STATIC IMPORTED)
set_target_properties(SDL2::SDL2-static PROPERTIES