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 \