Re: [8] Review request for 7162111 change tests run in headless mode [macosx]

2012-07-13 Thread Jason Uh
Anthony and Alan, Thanks for your comments. I've reverted the changes to CommonSetup.sh so that XToolkit is no longer forced. Tests still pass. Please see the new webrev: http://cr.openjdk.java.net/~juh/7162111/webrev.01/ Thanks, Jason On 06/25/2012 06:19 AM, Anthony Petrov wrote: Hi

Re: [8] Review request for 7162111 change tests run in headless mode [macosx]

2012-07-11 Thread Alan Bateman
On 02/07/2012 15:18, Jason Uh wrote: Anthony and Alan, Thanks for your comments. I've reverted the changes to CommonSetup.sh so that XToolkit is no longer forced. Tests still pass. Please see the new webrev: http://cr.openjdk.java.net/~juh/7162111/webrev.01/ Thanks, Jason This looks okay

Re: [8] Review request for 7162111 change tests run in headless mode [macosx]

2012-07-02 Thread Anthony Petrov
Looks fine to me. -- best regards, Anthony On 07/02/12 18:18, Jason Uh wrote: Anthony and Alan, Thanks for your comments. I've reverted the changes to CommonSetup.sh so that XToolkit is no longer forced. Tests still pass. Please see the new webrev:

Re: [8] Review request for 7162111 change tests run in headless mode [macosx]

2012-06-25 Thread Anthony Petrov
Hi Alan and Jason, On 06/23/12 11:28, Alan Bateman wrote: On 23/06/2012 02:01, Jason Uh wrote: This fix was for regression tests failing on Mac OS X on remotely executed environments. The changed tests now run in headless mode and have been taken off the Problem List. Webrev:

Re: [8] Review request for 7162111 change tests run in headless mode [macosx]

2012-06-23 Thread Alan Bateman
On 23/06/2012 02:01, Jason Uh wrote: Hi, This fix was for regression tests failing on Mac OS X on remotely executed environments. The changed tests now run in headless mode and have been taken off the Problem List. Webrev: http://cr.openjdk.java.net/~juh/7162111/webrev.00/ The CR: