Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-11 Thread xiaofeng Young
;> >>>> -Hamlin >>>> >>>> On 2018/10/11 11:08 AM, Igor Ignatyev wrote: >>>>> Hi Hamlin, >>>>> >>>>> as ModuleTargetHelper uses non-public API, I'd prefer not to have >>>>> in a common test library, and 82

Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-11 Thread Igor Ignatyev
; >>>>> On 2018/10/11 11:08 AM, Igor Ignatyev wrote: >>>>>> Hi Hamlin, >>>>>> >>>>>> as ModuleTargetHelper uses non-public API, I'd prefer not to have in a >>>>>> common test library, and 8211976 s

Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-11 Thread Hamlin Li
ed to put it into the library instead? Thanks, -- Igor - Original Message - From: huaming...@oracle.com <mailto:huaming...@oracle.com> To: core-libs-dev@openjdk.java.net <mailto:core-libs-dev@openjdk.java.net> Sent: Wednesday, October 10, 2018 7:40:34 PM GMT -08:00 US/C

Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-11 Thread Igor Ignatyev
211976 suggests moving it closer to tests. could >>>> you please explain why you decided to put it into the library instead? >>>> >>>> Thanks, >>>> -- Igor >>>> >>>> - Original Message - >>>> From:

Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-11 Thread Alan Bateman
On 11/10/2018 04:08, Igor Ignatyev wrote: Hi Hamlin, as ModuleTargetHelper uses non-public API, I'd prefer not to have in a common test library, and 8211976 suggests moving it closer to tests. could you please explain why you decided to put it into the library instead? Right,

Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-11 Thread Hamlin Li
Igor - Original Message - From: huaming...@oracle.com <mailto:huaming...@oracle.com> To: core-libs-dev@openjdk.java.net <mailto:core-libs-dev@openjdk.java.net> Sent: Wednesday, October 10, 2018 7:40:34 PM GMT -08:00 US/Canada Pacific Subject: RFR of JDK-8186610,move ModuleUtils t

Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-10 Thread Igor Ignatyev
to put it into the library instead? >> >> Thanks, >> -- Igor >> >> - Original Message - >> From: huaming...@oracle.com >> To: core-libs-dev@openjdk.java.net >> Sent: Wednesday, October 10, 2018 7:40:34 PM GMT -08:00 US/Canada Pacific >> Subj

Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-10 Thread Hamlin Li
, October 10, 2018 7:40:34 PM GMT -08:00 US/Canada Pacific Subject: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary Would you please review the following patch? bug:   https://bugs.openjdk.java.net/browse/JDK-8186610   https://bugs.openjdk.java.net/browse/JDK-8211976 webrev: http

Re: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-10 Thread Igor Ignatyev
...@oracle.com To: core-libs-dev@openjdk.java.net Sent: Wednesday, October 10, 2018 7:40:34 PM GMT -08:00 US/Canada Pacific Subject: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary Would you please review the following patch? bug:   https://bugs.openjdk.java.net/browse/JDK-8186610

RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

2018-10-10 Thread Hamlin Li
Would you please review the following patch? bug:   https://bugs.openjdk.java.net/browse/JDK-8186610   https://bugs.openjdk.java.net/browse/JDK-8211976 webrev: http://cr.openjdk.java.net/~mli/8186610/ Thank you -Hamlin