[gentoo-commits] repo/gentoo:master commit in: dev-java/ant-core/files/, dev-java/ant-core/

[Georgy Yakovlev]

commit:     a8720a4d009fcfa5a258244533a9f51a63f468b0
Author:     Georgy Yakovlev <gyakovlev <AT> gentoo <DOT> org>
AuthorDate: Tue Jan 21 23:44:14 2020 +0000
Commit:     Georgy Yakovlev <gyakovlev <AT> gentoo <DOT> org>
CommitDate: Wed Jan 22 00:22:31 2020 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a8720a4d

dev-java/ant-core: rebvump, add cmdline handling patch

Bug: https://bugs.gentoo.org/698876
Package-Manager: Portage-2.3.84, Repoman-2.3.16
Signed-off-by: Georgy Yakovlev <gyakovlev <AT> gentoo.org>

 dev-java/ant-core/ant-core-1.10.7-r1.ebuild       | 110 ++++++++++++++++++
 dev-java/ant-core/files/1.10.7-cmdline-args.patch | 135 ++++++++++++++++++++++
 2 files changed, 245 insertions(+)

diff --git a/dev-java/ant-core/ant-core-1.10.7-r1.ebuild b/dev-java/ant-core/ant-core-1.10.7-r1.ebuild
new file mode 100644
index 00000000000..64a382b259f
--- /dev/null
+++ b/dev-java/ant-core/ant-core-1.10.7-r1.ebuild
@@ -0,0 +1,110 @@
+# Copyright 1999-2020 Gentoo Authors
+# Distributed under the terms of the GNU General Public License v2
+
+EAPI=7
+
+# Don't depend on itself.
+JAVA_ANT_DISABLE_ANT_CORE_DEP="true"
+
+# Rewriting build.xml files for the testcases has no use at the moment.
+JAVA_PKG_BSFIX_ALL="no"
+JAVA_PKG_IUSE="doc source"
+
+inherit eutils java-pkg-2 java-ant-2 prefix
+
+MY_P="apache-ant-${PV}"
+
+DESCRIPTION="Java-based build tool similar to 'make' that uses XML configuration files"
+HOMEPAGE="https://ant.apache.org/"
+SRC_URI="https://archive.apache.org/dist/ant/source/${MY_P}-src.tar.bz2
+	https://dev.gentoo.org/~fordfrog/distfiles/ant-${PV}-gentoo.tar.bz2"
+
+LICENSE="Apache-2.0"
+SLOT="0"
+KEYWORDS="~amd64 ~arm ~arm64 ~ppc64 ~x86 ~ppc-aix ~amd64-linux ~x86-linux ~ppc-macos ~x64-macos ~x86-macos ~sparc-solaris ~sparc64-solaris ~x64-solaris ~x86-solaris"
+
+CDEPEND=">=virtual/jdk-1.8:*"
+DEPEND="${CDEPEND}"
+RDEPEND="${CDEPEND}"
+
+S="${WORKDIR}/${MY_P}"
+
+RESTRICT="test"
+
+PATCHES=( 
+	"${FILESDIR}/${PV}"-cmdline-args.patch
+	"${WORKDIR}/${PV}-build.patch" 
+	"${WORKDIR}/${PV}-launch.patch"
+)
+
+src_prepare() {
+	default
+
+	eprefixify "${S}/src/script/ant"
+
+	# Fixes bug 556008.
+	java-ant_xml-rewrite -f build.xml \
+		-c -e javadoc \
+		-a failonerror \
+		-v "false"
+
+	# See bug #196080 for more details.
+	java-ant_bsfix_one build.xml
+	java-pkg-2_src_prepare
+
+	# Remove JDK9+ stuff
+	einfo "Removing JDK9+ classes (Jmod and Link)"
+	rm "${S}"/src/main/org/apache/tools/ant/taskdefs/modules/{Jmod,Link}.java
+}
+
+src_compile() {
+	export ANT_HOME=""
+	# Avoid error message that package ant-core was not found
+	export ANT_TASKS="none"
+
+	local bsyscp
+
+	# This ensures that when building ant with bootstrapped ant,
+	# only the source is used for resolving references, and not
+	# the classes in bootstrapped ant but jikes in kaffe has issues with this...
+	if ! java-pkg_current-vm-matches kaffe; then
+		bsyscp="-Dbuild.sysclasspath=ignore"
+	fi
+
+	CLASSPATH="$(java-config -t)" ./build.sh ${bsyscp} jars dist-internal \
+		$(use_doc javadocs) || die "build failed"
+}
+
+src_install() {
+	dodir /usr/share/ant/lib
+
+	for jar in ant.jar ant-bootstrap.jar ant-launcher.jar ; do
+		java-pkg_dojar build/lib/${jar}
+		dosym ../../${PN}/lib/${jar} /usr/share/ant/lib/${jar}
+	done
+
+	dobin src/script/ant
+
+	dodir /usr/share/${PN}/bin
+	for each in antRun antRun.pl runant.pl runant.py ; do
+		dobin "${S}/src/script/${each}"
+		dosym ../../../bin/${each} /usr/share/${PN}/bin/${each}
+	done
+	dosym ../${PN}/bin /usr/share/ant/bin
+
+	insinto /usr/share/${PN}
+	doins -r dist/etc
+	dosym ../${PN}/etc /usr/share/ant/etc
+
+	echo "ANT_HOME=\"${EPREFIX}/usr/share/ant\"" > "${T}/20ant"
+	doenvd "${T}/20ant"
+
+	dodoc NOTICE README WHATSNEW KEYS
+
+	if use doc; then
+		dodoc -r manual/*
+		java-pkg_dojavadoc --symlink manual/api build/javadocs
+	fi
+
+	use source && java-pkg_dosrc src/main/*
+}

diff --git a/dev-java/ant-core/files/1.10.7-cmdline-args.patch b/dev-java/ant-core/files/1.10.7-cmdline-args.patch
new file mode 100644
index 00000000000..1be6f0bc5e7
--- /dev/null
+++ b/dev-java/ant-core/files/1.10.7-cmdline-args.patch
@@ -0,0 +1,135 @@
+From 729692d37a72c84998cfc65a6da6e078bbe0910a Mon Sep 17 00:00:00 2001
+From: sergiys <sergiys@××××××.com>
+Date: Wed, 23 Oct 2019 13:24:19 -0700
+Subject: [PATCH] Fix regression introduced by commit "Use commandline argument
+ file for all options, but -J, for case javac"
+
+---
+ .../ant/taskdefs/compilers/JavacExternal.java | 35 +++++++++-------
+ .../taskdefs/compilers/JavacExternalTest.java | 42 +++++++++++++++++++
+ 2 files changed, 62 insertions(+), 15 deletions(-)
+
+diff --git a/src/main/org/apache/tools/ant/taskdefs/compilers/JavacExternal.java b/src/main/org/apache/tools/ant/taskdefs/compilers/JavacExternal.java
+index 9569c7fe8f..637e9bdf6c 100644
+--- a/src/main/org/apache/tools/ant/taskdefs/compilers/JavacExternal.java
++++ b/src/main/org/apache/tools/ant/taskdefs/compilers/JavacExternal.java
+@@ -66,7 +66,7 @@ public boolean execute() throws BuildException {
+         int firstFileName;
+ 
+         if (assumeJava1_2Plus()) {
+-            firstFileName = moveJOptionsToBeginning(commandLine);
++            firstFileName = moveArgFileEligibleOptionsToEnd(commandLine);
+         } else {
+             firstFileName = -1;
+         }
+@@ -77,31 +77,36 @@ public boolean execute() throws BuildException {
+     }
+ 
+     /**
+-     * Moves all -J arguments to the beginning
+-     * So that all command line arguments could be written to file, but -J
++     * Moves all -J and @argfiles arguments to the beginning
++     * So that all command line arguments could be written to file, but -J and @argfile
+      * As per javac documentation:
+      *      you can specify one or more files that contain arguments to the javac command (except -J options)
+      * @param commandLine command line to process
+-     * @return int index of first non -J argument
++     * @return int index of first argument that could be put into argfile
+      */
+-    private int moveJOptionsToBeginning(String[] commandLine) {
+-        int nonJArgumentIdx = 1; // 0 for javac executable
+-        while(nonJArgumentIdx < commandLine.length && commandLine[nonJArgumentIdx].startsWith("-J")) {
+-            nonJArgumentIdx++;
++    private int moveArgFileEligibleOptionsToEnd(String[] commandLine) {
++        int nonArgFileOptionIdx = 1; // 0 for javac executable
++        while(nonArgFileOptionIdx < commandLine.length &&
++                !isArgFileEligible(commandLine[nonArgFileOptionIdx])) {
++            nonArgFileOptionIdx++;
+         }
+ 
+-        for(int i = nonJArgumentIdx + 1; i < commandLine.length; i++) {
+-            if (commandLine[i].startsWith("-J")) {
+-                String jArgument = commandLine[i];
+-                for(int j = i - 1; j >= nonJArgumentIdx; j--) {
++        for(int i = nonArgFileOptionIdx + 1; i < commandLine.length; i++) {
++            if (!isArgFileEligible(commandLine[i])) {
++                String option = commandLine[i];
++                for(int j = i - 1; j >= nonArgFileOptionIdx; j--) {
+                     commandLine[j + 1] = commandLine[j];
+                 }
+-                commandLine[nonJArgumentIdx] = jArgument;
+-                nonJArgumentIdx++;
++                commandLine[nonArgFileOptionIdx] = option;
++                nonArgFileOptionIdx++;
+             }
+         }
+ 
+-        return nonJArgumentIdx;
++        return nonArgFileOptionIdx;
++    }
++
++    private static boolean isArgFileEligible(String option) {
++        return !(option.startsWith("-J") || option.startsWith("@"));
+     }
+ 
+     /**
+diff --git a/src/tests/junit/org/apache/tools/ant/taskdefs/compilers/JavacExternalTest.java b/src/tests/junit/org/apache/tools/ant/taskdefs/compilers/JavacExternalTest.java
+index 53aac6377c..a893fb49c6 100644
+--- a/src/tests/junit/org/apache/tools/ant/taskdefs/compilers/JavacExternalTest.java
++++ b/src/tests/junit/org/apache/tools/ant/taskdefs/compilers/JavacExternalTest.java
+@@ -28,6 +28,7 @@
+ import java.io.File;
+ import java.io.IOException;
+ import java.util.Arrays;
++import java.util.stream.Stream;
+ 
+ import static org.junit.Assert.assertEquals;
+ import static org.junit.Assert.assertTrue;
+@@ -131,6 +132,47 @@ public void allJOptionsAreMovedToBeginning() throws Exception {
+         }
+     }
+ 
++    @Test
++    public void argFileOptionIsMovedToBeginning() throws Exception {
++        final File workDir = createWorkDir("testSMC");
++        try {
++            final File src = new File(workDir, "src");
++            src.mkdir();
++            createFile(src, "org/apache/ant/tests/J1.java");
++            createFile(src, "org/apache/ant/tests/J2.java");
++            final File modules = new File(workDir, "modules");
++            modules.mkdir();
++            final Project prj = new Project();
++            prj.setBaseDir(workDir);
++            final Javac javac = new Javac();
++            javac.setProject(prj);
++            final Commandline[] cmd = new Commandline[1];
++            final TestJavacExternal impl = new TestJavacExternal();
++            final Path srcPath = new Path(prj);
++            srcPath.setLocation(src);
++            javac.setSrcdir(srcPath);
++            javac.createModulepath().setLocation(modules);
++            javac.setSource("9");
++            javac.setTarget("9");
++            javac.setFork(true);
++            javac.setMemoryInitialSize("80m");
++            javac.setExecutable("javacExecutable");
++            javac.add(impl);
++            javac.createCompilerArg().setValue("-g");
++            javac.createCompilerArg().setValue("@/home/my-compiler.args");
++            javac.execute();
++            assertEquals("javacExecutable", impl.getArgs()[0]);
++            assertEquals("-J-Xms80m", impl.getArgs()[1]);
++            assertEquals("@/home/my-compiler.args", impl.getArgs()[2]);
++            assertTrue(Stream.of(impl.getArgs()).anyMatch(x -> x.equals("-g")));
++            assertTrue(impl.getArgs()[impl.getArgs().length - 2].endsWith("J1.java"));
++            assertTrue(impl.getArgs()[impl.getArgs().length - 1].endsWith("J2.java"));
++            assertEquals(3, impl.getFirstFileName());
++        } finally {
++            delete(workDir);
++        }
++    }
++
+     private File createWorkDir(String testName) {
+         final File tmp = new File(System.getProperty("java.io.tmpdir"));   //NOI18N
+         final File destDir = new File(tmp, String.format("%s%s%d",