Add custom JNI_INCLUDE_DIRS for Windows only

Signed-off-by: utzcoz <utzcoz@outlook.com>
utzcoz · Dec 6, 2021 · 29bad52 · 29bad52
1 parent f40cea1
commit 29bad52
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/nativeruntime/cpp/CMakeLists.txt b/nativeruntime/cpp/CMakeLists.txt
@@ -8,7 +8,15 @@ set (CMAKE_CXX_STANDARD 17)
 
 project(nativeruntime)
 
-find_package(JNI REQUIRED)
+if (WIN32)
+  if(NOT DEFINED ENV{JAVA_HOME})
+    message(FATAL_ERROR "JAVA_HOME is required in Windows")
+  endif()
+  # find_package JNI is broken on Windows, manually include header files
+  set(JNI_INCLUDE_DIRS "$ENV{JAVA_HOME}/include" "$ENV{JAVA_HOME}/include/win32")
+else()
+  find_package(JNI REQUIRED)
+endif()
 
 set(ANDROID_SQLITE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/../external/sqlite")