CMake: clean up HIDAPI build

CMake/FindHIDAPI.cmake

@@ -6,4 +6,12 @@ set(HIDAPI_INCLUDE_DIRS ${HIDAPI_INCLUDE_DIR})
 include(FindPackageHandleStandardArgs)
 find_package_handle_standard_args(HIDAPI DEFAULT_MSG HIDAPI_LIBRARY HIDAPI_INCLUDE_DIR)
 
+if(HIDAPI_FOUND AND NOT TARGET Hidapi::Hidapi)
+    add_library(Hidapi::Hidapi UNKNOWN IMPORTED)
+    set_target_properties(Hidapi::Hidapi PROPERTIES
+      IMPORTED_LOCATION ${HIDAPI_LIBRARIES}
+      INTERFACE_INCLUDE_DIRECTORIES ${HIDAPI_INCLUDE_DIRS}
+    )
+endif()
+
 mark_as_advanced(HIDAPI_INCLUDE_DIR HIDAPI_LIBRARY)

CMakeLists.txt

@@ -752,39 +752,9 @@ list(APPEND LIBS ${ICONV_LIBRARIES})
 
 if(NOT ANDROID)
   find_package(HIDAPI)
-  if(HIDAPI_FOUND)
-    message(STATUS "Using shared ${HIDAPI_LIBRARIES} ${HIDAPI_VERSION}")
-    include_directories(${HIDAPI_INCLUDE_DIRS})
-    list(APPEND LIBS ${HIDAPI_LIBRARIES})
-  else()
-    set(HIDAPI_FOUND 1)
-    include_directories(Externals/hidapi/hidapi)
-    if(APPLE)
-      message(STATUS "Using static hidapi from Externals")
-      add_subdirectory(Externals/hidapi/mac)
-      list(APPEND LIBS hidapi)
-    elseif(${CMAKE_SYSTEM_NAME} STREQUAL "Linux")
-      find_package(Libudev)
-      if(LIBUDEV_FOUND)
-        message(STATUS "Using static hidapi-hidraw from Externals")
-        add_subdirectory(Externals/hidapi/linux)
-        list(APPEND LIBS hidapi-hidraw udev)
-      else()
-        message(STATUS "Could not find udev, disabling hidapi")
-        set(HIDAPI_FOUND 0)
-      endif()
-    elseif(MSVC)
-      message(STATUS "Using static hidapi-hid from Externals")
-      add_subdirectory(Externals/hidapi/windows)
-      list(APPEND LIBS hidapi-hid)
-    else()
-      message(STATUS "Using static hidapi-libusb from Externals")
-      add_subdirectory(Externals/hidapi/libusb)
-      list(APPEND LIBS hidapi-libusb)
-    endif()
-  endif()
-  if(HIDAPI_FOUND)
-    add_definitions(-DHAVE_HIDAPI=1)
+  if(NOT HIDAPI_FOUND)
+    message(STATUS "Using static HIDAPI from Externals")
+    add_subdirectory(Externals/hidapi EXCLUDE_FROM_ALL)
   endif()
 endif()
 

Externals/hidapi/CMakeLists.txt

@@ -0,0 +1,20 @@
+project(hidapi)
+
+add_library(hidapi STATIC hidapi/hidapi.h)
+target_include_directories(hidapi PUBLIC hidapi)
+
+if(APPLE)
+  target_sources(hidapi PRIVATE mac/hid.c)
+elseif(MSVC)
+  target_sources(hidapi PRIVATE windows/hid.c)
+else()
+  find_package(Libudev)
+  if(CMAKE_SYSTEM_NAME STREQUAL "Linux" AND LIBUDEV_FOUND)
+    target_sources(hidapi PRIVATE linux/hid.c)
+    target_link_libraries(hidapi PRIVATE udev)
+  else()
+    target_sources(hidapi PRIVATE libusb/hid.c)
+  endif()
+endif()
+
+add_library(Hidapi::Hidapi ALIAS hidapi)