Bug 179630 - Compiler parsing tests fail with IBM J2SE 1.4.2, 5.0, and 6.0 (early access)
Summary: Compiler parsing tests fail with IBM J2SE 1.4.2, 5.0, and 6.0 (early access)
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3   Edit
Hardware: Power PC Linux
: P3 normal (vote)
Target Milestone: 3.3 M7   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-03-27 21:19 EDT by Corey Ashford CLA
Modified: 2007-04-27 09:45 EDT (History)
0 users

See Also:

Attachments
Fixes getJavaClassLibs for IBM JRE (2.13 KB, patch)
2007-03-27 21:22 EDT, Corey Ashford CLA 		no flags Details | Diff
Proposed fix (72.52 KB, patch)
2007-03-29 22:49 EDT, Olivier Thomann CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Corey Ashford CLA 2007-03-27 21:19:23 EDT 
Build ID: I20070323-1616

Steps To Reproduce:
1. Set up Eclipse 3.3M6 with the IBM J2SE 1.4.2, 5.0, or 6.0
2. Run the Automated test suite in org.eclipse.jdt.core.tests.compiler
3. Note the four errors in the test027 for compliance 1.3, 1.4, 1.5, and 1.6.


More information:
If you are using IBM J2SE 6.0, you will see many more errors than just the four tests above.

All of these errors are of the "Unable to resolve java.lang.Object" and "Unable to resolve org.xml. ..."

I have debugged these test cases and have found that the cause of the error is in org.eclipse.jdt.core.tests.util.Util in the method getJavaClassLibs, where it is missing adding the required jar files for these tests.

The jar files that are needed vary by IBM JRE release, unfortunately, complicating the code.

I will attach a patch that solves the problem for the three IBM JRE releases.
Comment 1 Corey Ashford CLA 2007-03-27 21:22:43 EDT 
Created attachment 62170 [details]
Fixes getJavaClassLibs for IBM JRE

Fixes for IBM J2SE 1.4.2, 5.0, 6.0 need to pass the jdt compiler parsing tests.
Comment 2 Olivier Thomann CLA 2007-03-29 22:49:35 EDT 
Created attachment 62455 [details]
Proposed fix

This patch is more generic. It checks if each entry of the classpaths exists before adding it.
Comment 3 Olivier Thomann CLA 2007-03-29 22:49:57 EDT 
I'll check it tomorrow on Linux. If it works fine, I'll release it.
Comment 4 Corey Ashford CLA 2007-03-29 23:26:08 EDT 
That sounds like a much better fix.  Thanks!
Comment 5 Olivier Thomann CLA 2007-03-31 14:45:20 EDT 
Released for 3.3M7.
Comment 6 Maxime Daniel CLA 2007-04-27 09:45:06 EDT 
Verified for 3.3 M7 using source code v_751.