Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-26 Thread Mandy Chung
oracle.com Sent: Tuesday, March 24, 2015 3:57:54 PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests This looks good, thank you for making these changes! A couple of comments that I don't feel need another webrev but should

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-26 Thread Yekaterina Kantserova
penjdk.java.net, alexandre.il...@oracle.com Sent: Friday, March 20, 2015 7:39:10 PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests I haven’t looked at the changes in detail, but please change the requiredVersion in TEST.ROOT to 4.1 b11

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-25 Thread Mandy Chung
PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests This looks good, thank you for making these changes! A couple of comments that I don't feel need another webrev but should be fixed before pushing. - copyrights on all the

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-25 Thread Lois Foltan
h...@oracle.com, alexandre.il...@oracle.com Sent: Tuesday, March 24, 2015 3:57:54 PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests This looks good, thank you for making these changes! A couple of comments that I don't feel need anoth

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-25 Thread Alexander Kulyakhtin
gt; >> Best regards, >> Alex >> >> - Original Message - >> From: staffan.lar...@oracle.com >> To: alexander.kulyakh...@oracle.com >> Cc: serviceability-dev@openjdk.java.net, alexandre.il...@oracle.com >> Sent: Friday, March 20, 2015 7:39:10 PM

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-24 Thread Alexander Kulyakhtin
.@oracle.com To: alexander.kulyakh...@oracle.com, lois.fol...@oracle.com Cc: hotspot-...@openjdk.java.net, serviceability-dev@openjdk.java.net, alexandre.il...@oracle.com Sent: Tuesday, March 24, 2015 6:07:27 PM GMT +04:00 Abu Dhabi / Muscat Subject: RE: RFR: JDK-8075586: add @modules as needed to the open ho

RE: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-24 Thread Christian Tornqvist
e.com Sent: Tuesday, March 24, 2015 3:57:54 PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests This looks good, thank you for making these changes! A couple of comments that I don't feel need another webrev but should be fixed

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-24 Thread Alexander Kulyakhtin
015 3:57:54 PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests This looks good, thank you for making these changes! A couple of comments that I don't feel need another webrev but should be fixed before pushing. - copyrights

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-24 Thread Lois Foltan
t, alexandre.il...@oracle.com Sent: Friday, March 20, 2015 7:39:10 PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests I haven’t looked at the changes in detail, but please change the requiredVersion in TEST.ROOT to 4.1 b11 as part

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-24 Thread Yekaterina Kantserova
:10 PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests I haven’t looked at the changes in detail, but please change the requiredVersion in TEST.ROOT to 4.1 b11 as part of this change. Thanks, /Staffan On 20 mar 2015, at 13:16

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-24 Thread Alexander Kulyakhtin
, Alex - Original Message - From: staffan.lar...@oracle.com To: alexander.kulyakh...@oracle.com Cc: serviceability-dev@openjdk.java.net, alexandre.il...@oracle.com Sent: Friday, March 20, 2015 7:39:10 PM GMT +04:00 Abu Dhabi / Muscat Subject: Re: RFR: JDK-8075586: add @modules as needed to the

Re: RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-20 Thread Staffan Larsen
I haven’t looked at the changes in detail, but please change the requiredVersion in TEST.ROOT to 4.1 b11 as part of this change. Thanks, /Staffan > On 20 mar 2015, at 13:16, Alexander Kulyakhtin > wrote: > > Hi, > > Could you, please, review the fix below. > > CR: https://bugs.openjdk.java.

RFR: JDK-8075586: add @modules as needed to the open hotspot tests

2015-03-20 Thread Alexander Kulyakhtin
Hi, Could you, please, review the fix below. CR: https://bugs.openjdk.java.net/browse/JDK-8075586 webrev: http://cr.openjdk.java.net/~tpivovarova/akulyakh/8075586/webrev.00/ The fix adds @modules keyword to the existing hotspot tests, as needed, so that the tests can access the required API whe