Re: Object classinfo Mauve tests failing under GNU Classpath

2012-12-04 Thread Pavel Tisnovsky
Hi Pekka,

thank you very much for your fix. You are right, it would be much better to
add tags to each test file - I can do this almost automatically for most test.
In case of native flag it would be AFAIK better to ignore this flag at all
- I'll do it too following week.

It would be nice to move from CVS to Mercurial/GIT and afaik there's already
CVS-GIT bridge setup by Mark Wielaard. I'm not sure who can do it, probably
Mark on Tom Tromey?

Thanks again for the patch,
Pavel


- Original Message -
 Hi Pavel,
 
 I noticed two issues with your current Mauve classinfo test cases
 with
 GNU Classpath.
 
   - I need the attached patch to make the Object classinfo tests run
 with JamVM/GNU Classpath; otherwise I see syntax errors while
 Mauve
 is trying to compile the test classes.
 
   - The tests in getMethods and getDeclaredMethods test cases are
 incompatible with GNU Classpath for two reasons: GNU Classpath
 has
 IllegalMonitorStateException in classfiles (which is arguably
 wrong)
 and methods are not native in GNU Classpath like they are with
 OpenJDK.
 
 P.S. Can we migrate Mauve source code to a git repository, please?
 Working with CVS is PITA.
 
 Pekka
 



Object classinfo Mauve tests failing under GNU Classpath

2012-11-07 Thread Pekka Enberg
Hi Pavel,

I noticed two issues with your current Mauve classinfo test cases with
GNU Classpath.

  - I need the attached patch to make the Object classinfo tests run
with JamVM/GNU Classpath; otherwise I see syntax errors while Mauve
is trying to compile the test classes.

  - The tests in getMethods and getDeclaredMethods test cases are
incompatible with GNU Classpath for two reasons: GNU Classpath has
IllegalMonitorStateException in classfiles (which is arguably wrong)
and methods are not native in GNU Classpath like they are with
OpenJDK.

P.S. Can we migrate Mauve source code to a git repository, please?
Working with CVS is PITA.

Pekka


0001-Annotate-test-cases-that-require-JDK-1.5.patch
Description: Binary data