Fixes linking to libSDLmain.a and libSDL.a in downstream packages.
find_package(SDL REQUIRED)
(with FindSDL.cmake provided by cmake) only sets SDL_LIBRARY and SDL_INCLUDE_DIR (without the trailing S). But catkin_package(DEPENDS <prefix>) looks for <prefix>_INCLUDE_DIRS and <prefix>_LIBRARIES. catkin emits a warning if neither of both is set for a dependency listed in DEPENDS, which should actually be treated as an error. Or catkin could accept multiple commonly used suffixes for include directories and libraries because apparently there is no standard. See related issue https://github.com/ros/catkin/issues/552.
In any case listing SDL in catkin_package(DEPS ...) is equivalent to listing include directories and libraries in INCLUDE_DIRS and LIBRARIES argument lists.
Another candidate to submit a patch upstream (#81).
Fixes linking to
libSDLmain.aandlibSDL.ain downstream packages.(with FindSDL.cmake provided by cmake) only sets
SDL_LIBRARYandSDL_INCLUDE_DIR(without the trailingS). Butcatkin_package(DEPENDS <prefix>)looks for<prefix>_INCLUDE_DIRSand<prefix>_LIBRARIES.catkinemits a warning if neither of both is set for a dependency listed inDEPENDS, which should actually be treated as an error. Orcatkincould accept multiple commonly used suffixes for include directories and libraries because apparently there is no standard. See related issue https://github.com/ros/catkin/issues/552.In any case listing
SDLincatkin_package(DEPS ...)is equivalent to listing include directories and libraries inINCLUDE_DIRSandLIBRARIESargument lists.Another candidate to submit a patch upstream (#81).