This is an automated email from the ASF dual-hosted git repository.
asekretenko pushed a commit to branch 1.8.x
in repository https://gitbox.apache.org/repos/asf/mesos.git
The following commit(s) were added to refs/heads/1.8.x by this push:
new 3728a1b Fixed build broken by a comment in the patch that fixes
`javah` race.
3728a1b is described below
commit 3728a1b74f13fe5a1185e8ecb08b8fbd7e65f98f
Author: Andrei Sekretenko <asekrete...@apache.org>
AuthorDate: Tue Jun 30 21:27:55 2020 +0200
Fixed build broken by a comment in the patch that fixes `javah` race.
This fixes build broken by b5a0168dcf6b912de8eb479472be344351a4c4e1.
---
src/Makefile.am | 15 ++++++++-------
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/src/Makefile.am b/src/Makefile.am
index 912632d..a7946d8 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -2090,14 +2090,15 @@ BUILT_SOURCES += $(nodist_libjava_la_SOURCES)
# * For 'javac -h' recipe, we substitute '_' with '/' to compute the source
path
# of the java file. Unlike 'javah', 'javac -h' requires java sources to
# generate headers.
+#
+# NOTE: javah has a race between concurrent runs trying to create
+# the same output directory:
+#
http://hg.openjdk.java.net/jdk8/jdk8/langtools/file/1ff9d5118aae/src/share/classes/com/sun/tools/javah/JavahTask.java#l475
+# Thus, we have to create the output directory here.
+#
+# TODO(asekretenko): Replace `javah` (deprecated as of JDK 9+)
+# with `javac` when switching to a newer JDK.
java/jni/%.h: $(MESOS_JAR)
- # NOTE: javah has a race between concurrent runs trying to create
- # the same output directory:
- #
http://hg.openjdk.java.net/jdk8/jdk8/langtools/file/1ff9d5118aae/src/share/classes/com/sun/tools/javah/JavahTask.java#l475
- # Thus, we have to create the output directory here.
- #
- # TODO(asekretenko): Replace `javah` (deprecated as of JDK 9+)
- # with `javac` when switching to a newer JDK.
mkdir -p java/jni
if HAS_JAVAH
$(JAVA_HOME)/bin/javah -d java/jni \
