Re: Review request: 8003851: MethodHandleNatives dependency on java.sql.DriverManager

2012-11-28 Thread Florian Weimer
On 11/27/2012 08:55 PM, Mandy Chung wrote: On 11/27/2012 11:46 AM, Alan Bateman wrote: This looks fine to me but when we move to module it means a reflection scope dependency on JDBC Right - this only removes the static dependency but the reflection scope dependency from base to JDBC remains.

Re: Review request: 8003851: MethodHandleNatives dependency on java.sql.DriverManager

2012-11-27 Thread David Holmes
On 28/11/2012 5:34 AM, Mandy Chung wrote: java.lang.invoke has a new dependency to java.sql.DriverManager and java.util.logging which are undesirable as they are currently not in jigsaw's base module. This also impacts the Profile work as JDBC is currently proposed in Compact2 and the JDBC depend

Re: Review request: 8003851: MethodHandleNatives dependency on java.sql.DriverManager

2012-11-27 Thread Mandy Chung
On 11/27/2012 11:46 AM, Alan Bateman wrote: On 27/11/2012 19:34, Mandy Chung wrote: java.lang.invoke has a new dependency to java.sql.DriverManager and java.util.logging which are undesirable as they are currently not in jigsaw's base module. This also impacts the Profile work as JDBC is curr

Re: Review request: 8003851: MethodHandleNatives dependency on java.sql.DriverManager

2012-11-27 Thread Alan Bateman
On 27/11/2012 19:34, Mandy Chung wrote: java.lang.invoke has a new dependency to java.sql.DriverManager and java.util.logging which are undesirable as they are currently not in jigsaw's base module. This also impacts the Profile work as JDBC is currently proposed in Compact2 and the JDBC depen

Review request: 8003851: MethodHandleNatives dependency on java.sql.DriverManager

2012-11-27 Thread Mandy Chung
java.lang.invoke has a new dependency to java.sql.DriverManager and java.util.logging which are undesirable as they are currently not in jigsaw's base module. This also impacts the Profile work as JDBC is currently proposed in Compact2 and the JDBC dependency means a reference from Compact1 to