harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tim Ellison <t.p.elli...@gmail.com>
Subject Re: [testing] Java 6 branch results so far
Date Thu, 02 Sep 2010 10:31:54 GMT
On 02/Sep/2010 11:21, Mark Hindess wrote:
> In message <4C7F538A.3060901@gmail.com>, Tim Ellison writes:
>> On 01/Sep/2010 18:39, Mark Hindess wrote:
>>> In message <AANLkTi==aLqfNzr8fFwVnuQ0=3zagvk7bO83ViMvm1rV@mail.gmail.com>,
>>> Catherine Hope writes:
>>>> I also see the failures in the javac tests, simply running:
>>>>> target/hdk/jdk/bin/javac HelloWorld.java
>>>> shows the exception:
>>>> Uncaught exception in main:
>>>> java.lang.NoClassDefFoundError:
>>>> javax/annotation/processing/ProcessingEnvironment ...
>>>>
>>>> The problem seems to be from commit 979981 - Apply java6 patch from
>>>> "[#HARMONY-6529] [jdktools]hdk javac only support compliance 1.5".
>>>> When I revert the hard-coded compliance level back to 1.5 then I don't
>>>> see the exception.
>>> I'll revert that commit if a second committer agrees?  (Until HARMONY-6437
>>> is completed.)
>>>
>>> I'm guessing the test failures come from the new unit tests included
>>> in a later merge, r984606, because I'm sure I would not have committed
>>> without running the tests.  I suppose I should mention that since I
>>> started doing single commit merges I no longer test merges unless they
>>> have conflicts.
>> +1 to reverting.  Going back to 1.5 is not ideal, but that's where we
>> were at the last milestone, and not running with a -1.6 clearly doesn't
>> advance things!
> 
> Done.
> 
>> BTW I didn't see you merge the SourceForge mirror patch forward to the
>> Java 6 branch.  Do you intend to do that? (or I can)
> 
> Done.  (I'd be happy to see someone else doing merges as well.)

Happy to do so, but with the "single commit merges" it would seem
appropriate for the original committer to do the merge ... however, I
really don't feel strongly about it and I'll willingly merge up stuff I
see going in.

Regards,
Tim

Mime
View raw message