TokTok / jvm-toxcore-c

JVM (Java/Scala/Kotlin) bindings to toxcore
https://toktok.ltd
GNU General Public License v3.0
40 stars 30 forks source link

chore: Only require jni.h to be found when searching for the JNI #96

Closed robinlinden closed 2 years ago

robinlinden commented 2 years ago

Before this patch, we would only add the jni.h include path if both JNI and AWT were found as FindJNI.cmake only sets JNI_FOUND if both JNI and AWT were found, requiring you to install openjdk-11-jdk instead of openjdk-11-jdk-headless for the build to succeed.

After this patch both the headless and the full jdk is fine.

See: https://cmake.org/cmake/help/v3.22/module/FindJNI.html


This change is Reviewable