Re-enabling builds of third-party containers (adoptium#225)

* Re-enable builds of third-party containers
* Update build.xml file with correct directory name
* Remove interactive mode from commandline
* Add test status
* Ignore more output files .gitignore

.gitignore

@@ -11,5 +11,10 @@ TestConfig/jvmTest.mk
 TestConfig/specToPlat.mk
 TestConfig/*.log
 TestConfig/tempTestTargetResult
-TestConfig/failedtargets.mk
-.DS_Store
+TestConfig/failedtargets.mk
+TestConfig/std.out
+systemtest_prereqs/*
+TestConfig/lib/*
+openjdk_regression/*.gz
+example/*.jar
+.DS_Store

TestConfig/scripts/build_test.xml

@@ -67,7 +67,6 @@
 					<property name="JAVA_VERSION" value="${JAVA_VERSION}" />
 					<property name="compiler.javac" value="${JAVAC}" />
 					<fileset dir="../../" includes="${build.list}" >
-						<exclude name="thirdparty_containers/build.xml" />
 					</fileset>
 				</subant>
 			</then>

TestConfig/scripts/testKitGen/makeGenTool/mkgen.pl

@@ -99,7 +99,7 @@ sub generateOnDir {
 	while ( my $entry = readdir $dir ) {
 		next if $entry eq '.' or $entry eq '..';
 		# temporarily exclude projects for CCM build (i.e., when JCL_VERSION is latest)
-		my $disabledDir = "thirdparty_containers";
+		my $disabledDir = "";
 		if (($JCL_VERSION ne "latest") or ($disabledDir !~ $entry )) {
 			my $projectDir  = $absolutedir . '/' . $entry;
 			if (( -f $projectDir ) && ( $entry eq 'playlist.xml' )) {

openjdk_regression/ProblemList_openjdk8.txt

@@ -231,6 +231,7 @@ com/sun/tools/attach/StartManagementAgent.java	115	generic-all
 
 # jdk_jdi
 com/sun/jdi/RedefineCrossEvent.java	227	macosx-all
+
 ############################################################################
 
 # jdk_util

thirdparty_containers/build.xml

@@ -0,0 +1,41 @@
+<?xml version="1.0"?>
+
+<!--
+# Licensed 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.
+-->
+
+<project name="thirdparty_containers" default="build" basedir=".">
+	<taskdef resource="net/sf/antcontrib/antlib.xml" />
+	<description>
+		Build external tests
+	</description>
+
+	<!-- set properties for this build -->
+	<property name="DEST" value="${BUILD_ROOT}/thirdparty_containers" />
+
+	<target name="init">
+		<mkdir dir="${DEST}" />
+	</target>
+
+	<target name="build" depends="dist">
+		<subant target="">
+			<fileset dir="." includes="*/build.xml" />
+		</subant>		
+	</target>
+
+	<target name="dist" depends="init">
+		<copy todir="${DEST}">
+			<fileset dir="." includes="*.mk"/>
+		</copy>
+	</target>
+</project>

thirdparty_containers/scala/build.xml

@@ -6,22 +6,22 @@
 	</description>
 
 	<!-- set properties for this build -->
-	<property name="DEST" value="${BUILD_ROOT}/containerTest/scala-test" />
+	<property name="DEST" value="${BUILD_ROOT}/thirdparty_containers/scala-test" />
 	<property name="dist" location="${DEST}/${JAVA_VERSION}" />
 	<property name="src" location="." />
 
 	<target name="init">
 		<mkdir dir="${dist}"/>
 	</target>
 
-	<target name="clean_image" depends="init" description="build scala test docker image">
+	<target name="clean_image" depends="init" description="clean scala test docker image if there is one">
 		<exec executable="docker">
 			<arg line="rmi -f adoptopenjdk-scala-test" />
 		</exec>
 	</target>
 
 	<target name="build_image" depends="clean_image" description="build scala test docker image">
-		<exec executable="docker">
+		<exec executable="docker"  failonerror="true">
 			<arg line="build -t adoptopenjdk-scala-test -f dockerfile/Dockerfile --pull ." />
 		</exec>
 	</target>

thirdparty_containers/scala/playlist.xml

@@ -15,7 +15,8 @@
 <playlist xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="../TestConfig/playlist.xsd">
 	<test>
 		<testCaseName>scala_test</testCaseName>
-		<command>docker run -it --rm -v $(JDK_HOME):/java  adoptopenjdk-scala-test:latest "jvm pos neg"</command>
+	<command>docker run --rm -v $(JDK_HOME):/java  adoptopenjdk-scala-test:latest "jvm pos neg"; \
+		$(TEST_STATUS)</command>
 		<subsets>
 			<subset>SE80</subset>
 			<subset>SE90</subset>