[9] RFR 8130665 "java/lang/SecurityManager/CheckSecurityProvider.java failing in jake on OS X"

2015-08-07 Thread Valerie Peng
Hi, Sean and build experts, Can you please review the fix for 8130665 "java/lang/SecurityManager/CheckSecurityProvider.java failing in jake on OS X"? The Apple provider fails to be instantiated in Jake due to the missing permission grants. However, it currently only fails in Jake workspace and

--disable-warnings-as-errors not working? Build failure on traditional Chinese Windows 8

2015-08-07 Thread Henry Jen
Hi, I am seeing following building errors on Traditional Chinese Windows 8, what makes me wondering is why disable-warnings-as-errors not silence this? Did I miss something obvious here? Following is in my configure.log, using configure arguments '--disable-warnings-as-errors --with-conf-name=r

Re: RFR: 8076470 - JEP 240: Remove the JVM TI hprof Agent

2015-08-07 Thread serguei.spit...@oracle.com
Ok. Thanks, Staffan! Serguei On 8/7/15 2:27 AM, Staffan Larsen wrote: On 7 aug 2015, at 11:24, serguei.spit...@oracle.com wrote: Hi Staffan, Looks good. Thanks! I'm re-posting the same question in this review: Q1: Should the folder jdk/src/demo/s

Re: RFR: 8076470 - JEP 240: Remove the JVM TI hprof Agent

2015-08-07 Thread serguei.spit...@oracle.com
Dmitry, On 8/7/15 2:31 AM, Dmitry Samersoff wrote: Serguei, Q1: Should the folder jdk/src/demo/share/jvmti/java_crw_demo also be removed? Yes. See https://bugs.openjdk.java.net/browse/JDK-8041639 Thank you for the comment. But, please, see also the related comment from Staffan. Thanks,

Re: RFR: 8076470 - JEP 240: Remove the JVM TI hprof Agent

2015-08-07 Thread Dmitry Samersoff
Serguei, > Q1: Should the folder jdk/src/demo/share/jvmti/java_crw_demo also be > removed? Yes. See https://bugs.openjdk.java.net/browse/JDK-8041639 -Dmitry On 2015-08-07 12:24, serguei.spit...@oracle.com wrote: > Hi Staffan, > > Looks good. > > I'm re-posting the same question in this review

Re: RFR: 8076470 - JEP 240: Remove the JVM TI hprof Agent

2015-08-07 Thread Staffan Larsen
> On 7 aug 2015, at 11:24, serguei.spit...@oracle.com wrote: > > Hi Staffan, > > Looks good. Thanks! > > I'm re-posting the same question in this review: > Q1: Should the folder jdk/src/demo/share/jvmti/java_crw_demo also be > removed? No, it is used by some of the demos (mtrace, minst, e

Re: RFR: 8076470 - JEP 240: Remove the JVM TI hprof Agent

2015-08-07 Thread serguei.spit...@oracle.com
Hi Staffan, Looks good. I'm re-posting the same question in this review: Q1: Should the folder jdk/src/demo/share/jvmti/java_crw_demo also be removed? Thanks, Serguei On 8/6/15 11:57 PM, Staffan Larsen wrote: Please review the following changes to remove the hprof JVMTI agent. There are