You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by bo...@apache.org on 2016/07/30 16:52:57 UTC
[1/5] ant git commit: additional tests for javah
Repository: ant
Updated Branches:
refs/heads/master b746f1e7d -> 5d6adaef8
additional tests for javah
this is really only a port of the junit test to antunit
Project: http://git-wip-us.apache.org/repos/asf/ant/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/1f9adc1d
Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/1f9adc1d
Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/1f9adc1d
Branch: refs/heads/master
Commit: 1f9adc1d5b3a2e040c9230b409facd55aceb0a8d
Parents: e74054f
Author: Stefan Bodewig <bo...@apache.org>
Authored: Sat Jul 30 18:34:32 2016 +0200
Committer: Stefan Bodewig <bo...@apache.org>
Committed: Sat Jul 30 18:34:32 2016 +0200
----------------------------------------------------------------------
.../antunit/taskdefs/optional/javah-test.xml | 48 ++++++++++++++++++++
1 file changed, 48 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ant/blob/1f9adc1d/src/tests/antunit/taskdefs/optional/javah-test.xml
----------------------------------------------------------------------
diff --git a/src/tests/antunit/taskdefs/optional/javah-test.xml b/src/tests/antunit/taskdefs/optional/javah-test.xml
index e85293e..a51eae3 100644
--- a/src/tests/antunit/taskdefs/optional/javah-test.xml
+++ b/src/tests/antunit/taskdefs/optional/javah-test.xml
@@ -62,4 +62,52 @@ public class Adapter implements JavahAdapter {
</javah>
<au:assertLogContains text="adapter called" />
</target>
+
+ <target name="-setupForRealJavahTests">
+ <condition property="sunjavah.present">
+ <or>
+ <available classname="com.sun.tools.javah.oldjavah.Main"/>
+ <available classname="com.sun.tools.javah.Main"/>
+ </or>
+ </condition>
+ <mkdir dir="${input}/org/example"/>
+ <mkdir dir="${output}/org/example"/>
+ <echo file="${input}/org/example/Foo.java">
+ <![CDATA[
+package org.example;
+public class Foo {
+ public Foo() {}
+ public native String bar(Object baz);
+}
+]]></echo>
+ <javac srcdir="${input}" destdir="${output}"/>
+ <presetdef name="javah-single">
+ <javah destdir="${output}">
+ <class name="org.example.Foo"/>
+ <classpath>
+ <pathelement location="${output}"/>
+ </classpath>
+ </javah>
+ </presetdef>
+ <presetdef name="javah-fileset">
+ <javah destdir="${output}">
+ <fileset dir="${output}">
+ <include name="**/*.class"/>
+ </fileset>
+ <classpath>
+ <pathelement location="${output}"/>
+ </classpath>
+ </javah>
+ </presetdef>
+ </target>
+
+ <target name="testSimpleCompile" depends="-setupForRealJavahTests">
+ <javah-single/>
+ <au:assertFileExists file="${output}/org_example_Foo.h"/>
+ </target>
+
+ <target name="testCompileUsingFileset" depends="-setupForRealJavahTests">
+ <javah-fileset/>
+ <au:assertFileExists file="${output}/org_example_Foo.h"/>
+ </target>
</project>
[2/5] ant git commit: implement a forking implementation for javah
Posted by bo...@apache.org.
implement a forking implementation for javah
Project: http://git-wip-us.apache.org/repos/asf/ant/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/7ebbdc99
Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/7ebbdc99
Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/7ebbdc99
Branch: refs/heads/master
Commit: 7ebbdc995c7ef0c78c63b078765dc714cdd80452
Parents: 1f9adc1
Author: Stefan Bodewig <bo...@apache.org>
Authored: Sat Jul 30 18:45:53 2016 +0200
Committer: Stefan Bodewig <bo...@apache.org>
Committed: Sat Jul 30 18:45:53 2016 +0200
----------------------------------------------------------------------
WHATSNEW | 3 +
manual/Tasks/javah.html | 10 ++-
.../taskdefs/optional/javah/ForkingJavah.java | 72 ++++++++++++++++++++
.../optional/javah/JavahAdapterFactory.java | 6 ++
.../ant/taskdefs/optional/javah/SunJavah.java | 2 +-
.../antunit/taskdefs/optional/javah-test.xml | 22 ++++++
6 files changed, 111 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ant/blob/7ebbdc99/WHATSNEW
----------------------------------------------------------------------
diff --git a/WHATSNEW b/WHATSNEW
index 869c804..019b8a4 100644
--- a/WHATSNEW
+++ b/WHATSNEW
@@ -64,6 +64,9 @@ Other changes:
magic.
Bugzilla Report 59860
+ * a new implementation "forking" has been added to <javah> and is
+ used as default when running on JDK9.
+
Changes from Ant 1.9.6 TO Ant 1.9.7
===================================
http://git-wip-us.apache.org/repos/asf/ant/blob/7ebbdc99/manual/Tasks/javah.html
----------------------------------------------------------------------
diff --git a/manual/Tasks/javah.html b/manual/Tasks/javah.html
index 34a7b87..0fdc885 100644
--- a/manual/Tasks/javah.html
+++ b/manual/Tasks/javah.html
@@ -37,12 +37,16 @@ systems are used.</p>
with the <code>implementation</code> attribute or a nested element. <a
name="implementationvalues">Here are the choices of the attribute</a>:</p>
<ul>
- <li>default - the default compiler (kaffeh or sun) for the platform.</li>
- <li>sun (the standard compiler of the JDK)</li>
+ <li>default - the default compiler for the platform.</li>
+ <li>sun (the standard compiler of the JDK) - default when not
+ running on Kaffee or gcj/gij or Java9.</li>
<li>kaffeh (the native standard compiler of <a href="http://www.kaffe.org" target="_top">Kaffe</a>)</li>
<li>gcjh (the native standard compiler
of <a href="http://gcc.gnu.org/java/"
- target="_top">gcj and gij</a>) <em>since Apache Ant 1.8.2</em></li>
+ target="_top">gcj and gij</a>) <em>since Apache Ant 1.8.2</em></li>
+ <li>forking - runs the javah executable via its command line
+ interface in a separate process. Default wehn running on
+ Java9. <em>since Ant 1.9.8</em></li>
</ul>
<p><b>Note:</b> if you are using this task to work on multiple files
http://git-wip-us.apache.org/repos/asf/ant/blob/7ebbdc99/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java b/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java
new file mode 100644
index 0000000..c12f594
--- /dev/null
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java
@@ -0,0 +1,72 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ */
+package org.apache.tools.ant.taskdefs.optional.javah;
+
+import java.io.IOException;
+
+import org.apache.tools.ant.BuildException;
+import org.apache.tools.ant.Project;
+import org.apache.tools.ant.taskdefs.Execute;
+import org.apache.tools.ant.taskdefs.LogStreamHandler;
+import org.apache.tools.ant.taskdefs.optional.Javah;
+import org.apache.tools.ant.types.Commandline;
+import org.apache.tools.ant.util.JavaEnvUtils;
+
+/**
+ * This implementation runs the javah executable in a separate process.
+ *
+ * @since Ant 1.9.8
+ */
+public class ForkingJavah implements JavahAdapter {
+
+ /**
+ * the name of this adapter for users to select
+ */
+ public static final String IMPLEMENTATION_NAME = "forking";
+
+ /**
+ * Performs the actual compilation.
+ * @param javah the calling javah task.
+ * @return true if the compilation was successful.
+ * @throws BuildException if there is an error.
+ */
+ public boolean compile(Javah javah) throws BuildException {
+ Commandline cmd = SunJavah.setupJavahCommand(javah);
+ Project project = javah.getProject();
+ String executable = JavaEnvUtils.getJdkExecutable("javah");
+ javah.log("Running " + executable, Project.MSG_VERBOSE);
+ cmd.setExecutable(executable);
+
+ //set up the args
+ String[] args = cmd.getCommandline();
+
+ try {
+ Execute exe = new Execute(new LogStreamHandler(javah,
+ Project.MSG_INFO,
+ Project.MSG_WARN));
+ exe.setAntRun(project);
+ exe.setWorkingDirectory(project.getBaseDir());
+ exe.setCommandline(args);
+ exe.execute();
+ return !exe.isFailure();
+ } catch (IOException exception) {
+ throw new BuildException("Error running " + executable
+ + " -maybe it is not on the path", exception);
+ }
+ }
+}
http://git-wip-us.apache.org/repos/asf/ant/blob/7ebbdc99/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java b/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java
index d98b427..cc4419c 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java
@@ -44,6 +44,8 @@ public class JavahAdapterFactory {
return Kaffeh.IMPLEMENTATION_NAME;
} else if (JavaEnvUtils.isGij()) {
return Gcjh.IMPLEMENTATION_NAME;
+ } else if (JavaEnvUtils.isAtLeastJavaVersion(JavaEnvUtils.JAVA_9)) {
+ return ForkingJavah.IMPLEMENTATION_NAME;
}
return SunJavah.IMPLEMENTATION_NAME;
}
@@ -87,6 +89,10 @@ public class JavahAdapterFactory {
} else if ((JavaEnvUtils.isGij() && choice == null)
|| Gcjh.IMPLEMENTATION_NAME.equals(choice)) {
return new Gcjh();
+ } else if ((JavaEnvUtils.isAtLeastJavaVersion(JavaEnvUtils.JAVA_9)
+ && choice == null)
+ || ForkingJavah.IMPLEMENTATION_NAME.equals(choice)) {
+ return new ForkingJavah();
} else if (SunJavah.IMPLEMENTATION_NAME.equals(choice)) {
return new SunJavah();
} else if (choice != null) {
http://git-wip-us.apache.org/repos/asf/ant/blob/7ebbdc99/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java b/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java
index 0b1655a..7911b17 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java
@@ -69,7 +69,7 @@ public class SunJavah implements JavahAdapter {
return ej.fork(javah) == 0;
}
- private Commandline setupJavahCommand(Javah javah) {
+ static Commandline setupJavahCommand(Javah javah) {
Commandline cmd = new Commandline();
if (javah.getDestdir() != null) {
http://git-wip-us.apache.org/repos/asf/ant/blob/7ebbdc99/src/tests/antunit/taskdefs/optional/javah-test.xml
----------------------------------------------------------------------
diff --git a/src/tests/antunit/taskdefs/optional/javah-test.xml b/src/tests/antunit/taskdefs/optional/javah-test.xml
index a51eae3..884523d 100644
--- a/src/tests/antunit/taskdefs/optional/javah-test.xml
+++ b/src/tests/antunit/taskdefs/optional/javah-test.xml
@@ -110,4 +110,26 @@ public class Foo {
<javah-fileset/>
<au:assertFileExists file="${output}/org_example_Foo.h"/>
</target>
+
+ <target name="testSimpleCompileSun" depends="-setupForRealJavahTests"
+ if="sunjavah.present">
+ <javah-single implementation="sun"/>
+ <au:assertFileExists file="${output}/org_example_Foo.h"/>
+ </target>
+
+ <target name="testCompileUsingFilesetSun" depends="-setupForRealJavahTests"
+ if="sunjavah.present">
+ <javah-fileset implementation="sun"/>
+ <au:assertFileExists file="${output}/org_example_Foo.h"/>
+ </target>
+
+ <target name="testSimpleCompileForking" depends="-setupForRealJavahTests">
+ <javah-single implementation="forking"/>
+ <au:assertFileExists file="${output}/org_example_Foo.h"/>
+ </target>
+
+ <target name="testCompileUsingFilesetForking" depends="-setupForRealJavahTests">
+ <javah-fileset implementation="forking"/>
+ <au:assertFileExists file="${output}/org_example_Foo.h"/>
+ </target>
</project>
[3/5] ant git commit: Merge branch '1.9.x'
Posted by bo...@apache.org.
Merge branch '1.9.x'
Project: http://git-wip-us.apache.org/repos/asf/ant/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/378fb22e
Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/378fb22e
Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/378fb22e
Branch: refs/heads/master
Commit: 378fb22e2a17474cf1d3d6a6a005cd830c724c14
Parents: b746f1e 7ebbdc9
Author: Stefan Bodewig <bo...@apache.org>
Authored: Sat Jul 30 18:49:17 2016 +0200
Committer: Stefan Bodewig <bo...@apache.org>
Committed: Sat Jul 30 18:49:17 2016 +0200
----------------------------------------------------------------------
WHATSNEW | 3 +
manual/Tasks/javah.html | 10 ++-
.../taskdefs/optional/javah/ForkingJavah.java | 72 ++++++++++++++++++++
.../optional/javah/JavahAdapterFactory.java | 6 ++
.../ant/taskdefs/optional/javah/SunJavah.java | 2 +-
.../antunit/taskdefs/optional/javah-test.xml | 70 +++++++++++++++++++
6 files changed, 159 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ant/blob/378fb22e/WHATSNEW
----------------------------------------------------------------------
[5/5] ant git commit: use forking javah by default
Posted by bo...@apache.org.
use forking javah by default
Project: http://git-wip-us.apache.org/repos/asf/ant/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/5d6adaef
Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/5d6adaef
Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/5d6adaef
Branch: refs/heads/master
Commit: 5d6adaef8038982c6b86491aa9b11cac64521fe6
Parents: 83bb121
Author: Stefan Bodewig <bo...@apache.org>
Authored: Sat Jul 30 18:51:48 2016 +0200
Committer: Stefan Bodewig <bo...@apache.org>
Committed: Sat Jul 30 18:51:48 2016 +0200
----------------------------------------------------------------------
WHATSNEW | 3 +++
.../tools/ant/taskdefs/optional/javah/ForkingJavah.java | 1 +
.../taskdefs/optional/javah/JavahAdapterFactory.java | 12 +++---------
3 files changed, 7 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ant/blob/5d6adaef/WHATSNEW
----------------------------------------------------------------------
diff --git a/WHATSNEW b/WHATSNEW
index 15c3759..e60ac6c 100644
--- a/WHATSNEW
+++ b/WHATSNEW
@@ -25,6 +25,9 @@ Changes that could break older environments:
be used when running on Java 9 since this option has been removed.
Bugzilla Report 59906
+ * <javah> will default to the "forking" implementation on Java8
+ as well.
+
Fixed bugs:
-----------
http://git-wip-us.apache.org/repos/asf/ant/blob/5d6adaef/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java b/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java
index c12f594..a537669 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/javah/ForkingJavah.java
@@ -45,6 +45,7 @@ public class ForkingJavah implements JavahAdapter {
* @return true if the compilation was successful.
* @throws BuildException if there is an error.
*/
+ @Override
public boolean compile(Javah javah) throws BuildException {
Commandline cmd = SunJavah.setupJavahCommand(javah);
Project project = javah.getProject();
http://git-wip-us.apache.org/repos/asf/ant/blob/5d6adaef/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java b/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java
index cc4419c..481a97d 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/javah/JavahAdapterFactory.java
@@ -44,10 +44,8 @@ public class JavahAdapterFactory {
return Kaffeh.IMPLEMENTATION_NAME;
} else if (JavaEnvUtils.isGij()) {
return Gcjh.IMPLEMENTATION_NAME;
- } else if (JavaEnvUtils.isAtLeastJavaVersion(JavaEnvUtils.JAVA_9)) {
- return ForkingJavah.IMPLEMENTATION_NAME;
}
- return SunJavah.IMPLEMENTATION_NAME;
+ return ForkingJavah.IMPLEMENTATION_NAME;
}
/**
@@ -89,9 +87,7 @@ public class JavahAdapterFactory {
} else if ((JavaEnvUtils.isGij() && choice == null)
|| Gcjh.IMPLEMENTATION_NAME.equals(choice)) {
return new Gcjh();
- } else if ((JavaEnvUtils.isAtLeastJavaVersion(JavaEnvUtils.JAVA_9)
- && choice == null)
- || ForkingJavah.IMPLEMENTATION_NAME.equals(choice)) {
+ } else if (ForkingJavah.IMPLEMENTATION_NAME.equals(choice)) {
return new ForkingJavah();
} else if (SunJavah.IMPLEMENTATION_NAME.equals(choice)) {
return new SunJavah();
@@ -102,9 +98,7 @@ public class JavahAdapterFactory {
.createClassLoader(classpath));
}
- // This default has been good enough until Ant 1.6.3, so stick
- // with it
- return new SunJavah();
+ return new ForkingJavah();
}
/**
[4/5] ant git commit: typo
Posted by bo...@apache.org.
typo
Project: http://git-wip-us.apache.org/repos/asf/ant/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/83bb1217
Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/83bb1217
Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/83bb1217
Branch: refs/heads/master
Commit: 83bb1217aa49833485e1524846ac0fd6bc5374e0
Parents: 378fb22
Author: Stefan Bodewig <bo...@apache.org>
Authored: Sat Jul 30 18:51:39 2016 +0200
Committer: Stefan Bodewig <bo...@apache.org>
Committed: Sat Jul 30 18:51:39 2016 +0200
----------------------------------------------------------------------
WHATSNEW | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ant/blob/83bb1217/WHATSNEW
----------------------------------------------------------------------
diff --git a/WHATSNEW b/WHATSNEW
index 884d9b5..15c3759 100644
--- a/WHATSNEW
+++ b/WHATSNEW
@@ -15,7 +15,7 @@ Changes that could break older environments:
same instance. This will now cause the build to fail.
Bugzilla Report 59402
- * <native2ascii> will default to the buitin implementation on Java8
+ * <native2ascii> will default to the builtin implementation on Java8
as well (sun isn't available for Java9+ anyway).
* The ant.java.version property will now hold the value "9" rather