Re: RFR 7191662: JCE providers should be located via ServiceLoader

2015-05-28 Thread Valerie Peng
Please find comments in line... On 5/27/2015 3:42 PM, Mandy Chung wrote: Valerie, Did you see my comment yesterday? http://mail.openjdk.java.net/pipermail/security-dev/2015-May/012254.html Yes, we exchanged emails after this above one. I will follow up your latest one later today.

Re: Status of ccache

2015-05-28 Thread Dan Smith
On May 25, 2015, at 2:38 AM, Erik Joelsson erik.joels...@oracle.com wrote: On 2015-05-22 17:47, Dan Smith wrote: JDK-8027584 disabled ccache by default, I gather because it doesn't work in Cygwin, and secondarily because of vague general problems with it. The documentation

RFR: 8081447: System JPEG builds include in-tree jpeglib.h, resulting in build failure

2015-05-28 Thread Andrew Hughes
Bug: https://bugs.openjdk.java.net/browse/JDK-8081447 Webrev: http://cr.openjdk.java.net/~andrew/8081447/webrev.01/ In testing a fix for [0], I found that system jpeg support seems to have bitrotted. Due to the movement of makefiles, it's hard to trace where this change occurred, but we have the

Re: AWT Dev RFR: 8081447: System JPEG builds include in-tree jpeglib.h, resulting in build failure

2015-05-28 Thread Phil Race
Looks OK to me. I've added 2d-dev (owner of the lib). I don't think this will cause any issue for closed builds but we'll address it if it does. -phil. On 5/28/15 7:25 PM, Andrew Hughes wrote: - Original Message - Bug: https://bugs.openjdk.java.net/browse/JDK-8081447 Webrev:

Re: RFR 7191662: JCE providers should be located via ServiceLoader

2015-05-28 Thread Alan Bateman
On 27/05/2015 23:42, Mandy Chung wrote: Valerie, Did you see my comment yesterday? http://mail.openjdk.java.net/pipermail/security-dev/2015-May/012254.html Since you have reverted the java.security to keep the classname, to avoid causing merge conflict to jimage refresh, let’s remove the

Re: RFR(S): 8081037: serviceability/sa/ tests time out on Windows

2015-05-28 Thread Yekaterina Kantserova
Hi, due to https://bugs.openjdk.java.net/browse/JDK-8081381 I wasn't able to push this fix. The problem is LingeredApp.java contains JDK 9 feature java.lang.Process.getPid() but the test library is compiled with JDK 8 today. This issue is not trivial to solve so I suggest a temporary fix to