This is an automated email from the ASF dual-hosted git repository.

asekretenko pushed a commit to branch 1.9.x
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/1.9.x by this push:
     new 7f5d1c6  Fixed build broken by a comment in the patch that fixes 
`javah` race.
7f5d1c6 is described below

commit 7f5d1c6efb60746c6fbd4dac9c59d5b7248e7fe2
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 c4f012f4fb82d38894c4a33e05de127f4788db02.
---
 src/Makefile.am | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index 11d0924..0a9baac 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -2101,14 +2101,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                              \

Reply via email to