Changeset 36531 for main/trunk

Timestamp:

2022-08-29T13:21:36+12:00 (20 months ago)

Author:

davidb

Message:

Changed to a more careful way of determining whether javah (used in older JDKs) should be used or not. Need to do this in a way that doesn't accidentally pick up on an older system-wide installed JDK (that has a javah), given that we use JAVA_HOME to precisely determine which java/javac that should be used

File:

• main/trunk/greenstone3/src/packages/javagdbm/java/Makefile.in (modified) (2 diffs)

main/trunk/greenstone3/src/packages/javagdbm/java/Makefile.in

@@ -8 +8 @@
 # 'javah' existed until JDK 1.7, transitional in JDK 1.8, then removed by JDK 1.9 (use 'javac -h' instead)
 # Makefile rules below test for 'javah' and use it if it found, otherwise runs with 'javac -h' version
-JAVAH = javah
+#
+# Since we're committed to JAVA_HOME being set, so jni.h header files can be found,
+# then we can lean on JAVA_HOME here to more precisely check for the existence of javah
+# within JAVA_HOME, and not accidentally pick up on another (older) JDK that is also
+# installed on the system, and does come with a 'javah' but is unlikely to be able
+# to cope with newer (e.g. JDK 11) version class files
+JAVAH = $(JAVA_HOME)/javah
 JAR = jar
 
@@ -29 +35 @@
     @echo Checking for javah ...
 #   $(eval found = $(shell which javah 2>/dev/null | wc -l))
-    $(eval found = $(shell command -v javah | wc -l))
+    $(eval found = $(shell command -v $(JAVAH) | wc -l))
     @if test "$(found)" = "1" ; then \
       echo "... found" ; \