Suggested backports of JDK 7 langtools fixes to OpenJDK 6

[Jonathan Gibbons]

Andrew John Hughes wrote:
> On 3 March 2010 19:10, Jonathan Gibbons <Jonathan.Gibbons at sun.com> wrote:
>   
>> Andrew John Hughes wrote:
>>
>> On 3 March 2010 17:24, Joseph D. Darcy <Joe.Darcy at sun.com> wrote:
>>
>>
>> Jonathan Gibbons wrote:
>>
>>
>> Martin Buchholz wrote:
>>
>>
>> On Wed, Mar 3, 2010 at 08:29, Andrew John Hughes
>> <gnu_andrew at member.fsf.org> wrote:
>>
>>
>>
>> On 3 March 2010 00:26, Joseph D. Darcy <Joe.Darcy at sun.com> wrote:
>>
>>
>>
>>
>>
>>               -es -esa \
>>
>>
>>
>> I think
>> -vmoption:-es -vmoption:-esa
>> was meant.
>>
>>
>>
>> Whoops,
>>
>> I think
>>    -ea -esa
>> was meant.
>>
>>
>>
>> Yes, sorry for the typo!  The options I meant are "-ea" and "-esa" to enable
>> assertions and enable system assertions, respectively.
>>
>> -Joe
>>
>>
>>
>> With the right options, two tests fail:
>>
>> FAILED: tools/javac/processing/environment/TestSourceVersion.java:
>> javac: invalid source release: 1.7
>> FAILED: tools/javac/unicode/Unmappable.java: Unmappable.java:37:
>> warning: unmappable character for encoding ascii String s =
>> "z\ufffdhler";
>>
>> The first seems a correct failure; javac on OpenJDK6 should not support 1.7.
>> I see different failures on 7:
>>
>> FAILED: tools/apt/Basics/print.sh: java.lang.AssertionError at
>> com.sun.mirror.util.SourceOrderDeclScanner$SourceOrderComparator.compareEqualPosition(SourceOrderDeclScanner.java:104)
>> FAILED: tools/apt/Scanners/scanner.sh: java.lang.AssertionError at
>> com.sun.mirror.util.SourceOrderDeclScanner$SourceOrderComparator.compareEqualPosition(SourceOrderDeclScanner.java:104)
>> FAILED: tools/javac/api/TestJavacTaskScanner.java:
>> java.lang.IllegalArgumentExceptionat
>> com.sun.tools.javac.util.AbstractDiagnosticFormatter.formatSource(AbstractDiagnosticFormatter.java:138)
>> Error:  tools/javac/processing/6499119/ClassProcessor.java: test
>> result: Error. Parse Exception: Bad option for compile: process
>> Error:
>> tools/javac/processing/environment/round/TestElementsAnnotatedWith.java:
>> Error. Parse Exception: Bad option for compile: process
>>
>>
>> This one (Bad option for compile: process) is indicative of needing a newer
>> version of jtreg.
>>
>> FAILED: tools/javac/treepostests/TreePosTest.java: java.lang.Error:
>> failed at TreePosTest.main(TreePosTest.java:118)
>> FAILED: tools/javac/EarlyAssert.java: java.lang.Error: Assertions are
>> not disabled after initialization as they should be.
>>
>>
>>     
>
> I'm not surprised.  There are still no source releases of jtharness
> (https://jtharness.dev.java.net/issues/show_bug.cgi?id=20).  A
> cut-down version is maintained in IcedTea, which reduces the bloated
> list of dependencies
> (https://jtharness.dev.java.net/jt_buildinfo.html) including the
> proprietary Java Communications 3.0 API.  This was originally imported
> by Mark Wielaard some time ago.  From test/jtreg/README in IcedTea:
>
> 'This is a special "merge" of jtharness and jtreg to be compiled and
> run against the OpenJDK included in IcedTea. Please ONLY use this for
> testing inside the IcedTea environment. This is NOT a general jtreg
> and/or jtharness (source) distribution. In particular this version
> does NOT include support for javax.help, javax.servlet, javax.comm,
> junit, ant or the sun.com.jct classes. It is just a bare bones version
> that can be compiled and run against a "bare" JDK without needing any
> external libraries.
>
> PLEASE DO NOT PACKAGE AND DISTRIBUTE SEPARATELY FROM ICEDTEA
>
> This version is based on:
>
> - jtharness-oss-4_1_3a-dev
> - jtreg-4_0-src-b02-15_oct_2008'
>
> As you can see, it's now quite dated and needs an update at some point.
>   


I don't understand the comment about no source releases of jtharness.  
It is open source with GPL2+classpath exception, available in Subversion 
repositories here: https://jtharness.dev.java.net/source/browse/jtharness/

-- Jon
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/jdk6-dev/attachments/20100303/9a981a4d/attachment.html