Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Include cofoja jar in classpath when testing release jars

-Even though we're no longer compiling/using contracts in tests,
 we still need the cofoja jar in the classpath when testing the
 release jars due to some bad behavior on the part of TestNG in
 not being able to handle missing annotation classes.

-We don't need to package the cofoja classes in the actual GATK
 jar, however (and we never have).
  • Loading branch information...
commit 3390fc7d67f18d73eb48235ca40b1a7b5df5f30f 1 parent e29b52b
@droazen droazen authored
Showing with 6 additions and 0 deletions.
  1. +6 −0 build.xml
View
6 build.xml
@@ -1315,6 +1315,7 @@
<fileset dir="${lib.dir}">
<include name="scala*.jar"/>
<include name="jna*.jar"/>
+ <include name="cofoja*.jar"/>
</fileset>
</path>
</path>
@@ -1332,6 +1333,11 @@
<pathelement location="${java.contracts.dir}" />
<pathelement location="${java.test.classes}" />
<pathelement location="${scala.test.classes}" />
+ <path>
+ <fileset dir="${lib.dir}">
+ <include name="cofoja*.jar"/>
+ </fileset>
+ </path>
</path>
<property name="testng.classpath" value="queue.binary.release.tests.classpath" />
</target>
Please sign in to comment.
Something went wrong with that request. Please try again.