Re: RFR [XS] - 8139026 hotspot.script cannot handle command-line arguments with spaces

2015-10-30 Thread Ioi Lam
On 10/30/15 10:58 AM, Christian Thalinger wrote: On Oct 25, 2015, at 10:34 PM, Ioi Lam wrote: Please review a very small fix: http://cr.openjdk.java.net/~iklam/8139026-hotspot-script-arg-quoating/ Looks good. I’m sure you asked, who is still using this script? Thanks

Re: RFR: JDK-8141051: Build test libs -source/-target 9

2015-10-30 Thread Vladimir Ivanov
Looks good. Thanks for fixing it! Best regards, Vladimir Ivanov On 10/30/15 3:51 PM, Erik Joelsson wrote: Hello, Here is a new patch for this where I did some more cleanup. Webrev: http://cr.openjdk.java.net/~erikj/8141051/webrev.01/ Bug: https://bugs.openjdk.java.net/browse/JDK-8141051

RFR: JDK-8141051: Build test libs -source/-target 9

2015-10-30 Thread Erik Joelsson
Hello, Here is a new patch for this where I did some more cleanup. Webrev: http://cr.openjdk.java.net/~erikj/8141051/webrev.01/ Bug: https://bugs.openjdk.java.net/browse/JDK-8141051 /Erik On 2015-10-29 15:48, Erik Joelsson wrote: Hello, On 2015-10-29 14:43, Vladimir Ivanov wrote: Hi,

Re: RFR: JDK-8141051: Build test libs -source/-target 9

2015-10-30 Thread Tim Bell
Erik Looks good to me as well. /Tim On 10/30/15 06:06, Vladimir Ivanov wrote: Looks good. Thanks for fixing it! Best regards, Vladimir Ivanov On 10/30/15 3:51 PM, Erik Joelsson wrote: Hello, Here is a new patch for this where I did some more cleanup. Webrev:

Re: RFR [XS] - 8139026 hotspot.script cannot handle command-line arguments with spaces

2015-10-30 Thread Christian Thalinger
> On Oct 25, 2015, at 10:34 PM, Ioi Lam wrote: > > Please review a very small fix: > > http://cr.openjdk.java.net/~iklam/8139026-hotspot-script-arg-quoating/ Looks good. I’m sure you asked, who is still using this script? > > Bug: hotspot/make/hotspot.script cannot