RFR: JDK-8067013: Rename the com.oracle.java.testlibary package (original) (raw)

Alexander Kulyakhtin alexander.kulyakhtin at oracle.com
Wed Apr 29 11:38:22 UTC 2015

David,

Thank you very much for your approval.

I've created https://bugs.openjdk.java.net/browse/JDK-8078962 to add the missing copyright to the said file and will fix it today.

Best regards, Alexander

----- Original Message ----- From: david.holmes at oracle.com To: alexander.kulyakhtin at oracle.com, hotspot-dev at openjdk.java.net Cc: serviceability-dev at openjdk.java.net Sent: Wednesday, April 29, 2015 2:23:00 PM GMT +03:00 Iraq Subject: Re: RFR: JDK-8067013: Rename the com.oracle.java.testlibary package

On 28/04/2015 11:09 PM, Alexander Kulyakhtin wrote:

Hi,

I've updated the webrev in accordance with David's comments. Could you, please, review the changes:

https://bugs.openjdk.java.net/browse/JDK-8067013: Rename the com.oracle.java.testlibary package Webrev: http://cr.openjdk.java.net/~akulyakh/8067013/webrev.01/index.html

Looks good.

Aside: test/compiler/c2/8005956/PolynomialRoot.java doesn't have an OpenJDK copyright header. (Added by JDK-8017510)

Thanks, David

Best regards, Alex

----- Original Message ----- From: alexander.kulyakhtin at oracle.com To: david.holmes at oracle.com, hotspot-dev at openjdk.java.net Cc: serviceability-dev at openjdk.java.net Sent: Tuesday, April 28, 2015 2:08:54 PM GMT +03:00 Iraq Subject: Re: RFR: JDK-8067013: Rename the com.oracle.java.testlibary package Hi David, Thank you very much for your comments. I presume this is a first step towards fixing: https://bugs.openjdk.java.net/browse/JDK-8075327 (merging the two test libraries) ? Yes, this is correct Did you use "hg rename" for that? The webrev suggests not as it shows No, I used hg add and hg delete. I'm going to fix this now by doing hg rename Copyright dates will need updating in many cases. I'm going to update the copyrights I'll send out the updated webrev as soon as I'm done. Best regards, Alex ----- Original Message ----- From: david.holmes at oracle.com To: alexander.kulyakhtin at oracle.com, serviceability-dev at openjdk.java.net, hotspot-dev at openjdk.java.net Sent: Monday, April 27, 2015 4:39:31 AM GMT +03:00 Iraq Subject: Re: RFR: JDK-8067013: Rename the com.oracle.java.testlibary package Hi Alex, On 25/04/2015 12:33 AM, Alexander Kulyakhtin wrote: Hi,

Could I, please, have a review of this tests-only change: https://bugs.openjdk.java.net/browse/JDK-8067013: Rename the com.oracle.java.testlibary package Webrev: http://cr.openjdk.java.net/~akulyakh/8067013/webrev.00/index.html The change renames com.oracle.java.testlibrary package to jdk.test.lib package:

The directory structure have been changed under the hotspot/test/testlibrary/ to provide for the package renaming, deleted and added files rather than moved ones. Everywhere in the hotspot/test files import statements have been changed as needed. Seems okay based on scanning through the patch. Copyright dates will need updating in many cases. Thanks, David Best regards, Alex

More information about the hotspot-dev mailing list