Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 351669 - PlainDriverDataSource requires Java 6 but Manifest says Java 5
Summary: PlainDriverDataSource requires Java 6 but Manifest says Java 5
Status: CLOSED FIXED
Alias: None
Product: Gemini.JPA
Classification: RT
Component: Core (show other bugs)
Version: 1.0.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 1.0.0 RC3   Edit
Assignee: Michael Keith CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-11 03:19 EDT by Gunnar Wagenknecht CLA
Modified: 2011-08-16 13:23 EDT (History)
0 users

See Also:

Attachments
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 Gunnar Wagenknecht CLA 2011-07-11 03:19:42 EDT 
PlainDriverDataSource uses "new SQLException(Throwable)" which is not available in Java 5. If I update the bundle manifest to Java 6 (Bundle-RequiredExecutionEnvironment: JavaSE-1.6) then everything compiles fine.
Comment 1 Michael Keith CLA 2011-07-11 14:51:51 EDT 
Yes, indeed. Thanks.
Changed to use SQLException(String,Throwable) method.
Comment 2 Gunnar Wagenknecht CLA 2011-07-14 06:40:22 EDT 
Mike, I'm not sure this constructor is available. In addition to a 1.6 JDK I also have a JDK 1.5.0_16 configured in my workspace. Eclipse will use this to compile.

The only constructors I see are:
 java.sql.SQLException.SQLException()
 java.sql.SQLException.SQLException(String)
 java.sql.SQLException.SQLException(String, String)
 java.sql.SQLException.SQLException(String, String, int)

Thus, it still doesn't compile against Java 5.
Comment 3 Michael Keith CLA 2011-07-15 15:03:25 EDT 
Strange. I thought I was looking at the Java SE 5 javadoc when I saw the (String,Throwable) method. Apparently I was looking at SE 6. 

Now that SE 7 is out I think a better answer is just to change the required EE to SE 6.
Comment 4 Michael Keith CLA 2011-08-16 13:23:43 EDT 
Changed dependency to SE 6.