#76 current head revision of emma does not build in gump

[Antoine Levy-Lambert]

Hi,

gump is an integration tool of the Apache Software Foundation. 2 gump instances try to build emma (head revision) from the head revision of the sources of emma and of its dependencies every night.

Build is failing currently with this message :

core.compile:
[javac] Compiling 175 source files to /x1/gump/public/workspace/emma/out/core/classes
[javac]
[javac] WARNING
[javac]
[javac] The -source switch defaults to 1.5 in JDK 1.5 and 1.6.
[javac] If you specify -target 1.2 you now must also specify -source 1.2.
[javac] Ant will implicitly add -source 1.2 for you. Please change your build file.
[javac] /x1/gump/public/workspace/emma/core/java12/com/vladium/emma/run/InstrClassLoader.java:346: reference to CodeSource is ambiguous, both method CodeSource(java.net.URL,java.security.cert.Certificate[]) in java.security.CodeSource and method CodeSource(java.net.URL,java.security.CodeSigner[]) in java.security.CodeSource match
[javac] final CodeSource csrc = new CodeSource (srcURL, null);
[javac] ^
[javac] 1 error

see this link :
http://vmgump.apache.org/gump/public/emma/emma/gump_work/build_emma_emma.html

would be cool if you would fix it.

This is under JDK 1.5

Regards,
Antoine

[Gus Heck]

The following patch will fix this problem (cause by http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4983021\)

Index: core/java12/com/vladium/emma/run/InstrClassLoader.java

RCS file: /cvsroot/emma/emma/core/java12/com/vladium/emma/run/InstrClassLoader.java,v
retrieving revision 1.1
diff -u -r1.1 InstrClassLoader.java
--- core/java12/com/vladium/emma/run/InstrClassLoader.java 21 Jun 2005 02:40:32 -0000 1.1
+++ core/java12/com/vladium/emma/run/InstrClassLoader.java 26 Jan 2010 21:58:03 -0000
@@ -343,7 +343,7 @@
// support ProtectionDomains with non-null class source URLs:
// [however, disable anything related to sealing or signing]

- final CodeSource csrc = new CodeSource (srcURL, null);
+ final CodeSource csrc = new CodeSource (srcURL, (java.security.cert.Certificate[])null);

// allow getPackage() to return non-null on the class we are about to
// define (however, don't bother emulating the original manifest info since
@@ -459,4 +459,4 @@
private static final URL [] EMPTY_URL_ARRAY = new URL [0];

} // end of class
-// ----------------------------------------------------------------------------
\ No newline at end of file
+// ----------------------------------------------------------------------------