Looks good for me.
You can add me as reviewer for the push.
Regards,
-uta
On 28.01.2013 19:38, Rob McKenna wrote:
Can I take this as your final approval? (also, does this apply to the
closed review?)
Thanks,
-Rob
On 25/01/13 20:43, Rob McKenna wrote:
Yup, all affected tests pass in both jprt and via an ssh session to a
mac.
-Rob
On 25/01/13 16:07, Alexey Utkin wrote:
Looks good. Did you test the fix in ssh session to Mac?
-uta
On 25.01.2013 19:37, Rob McKenna wrote:
Had a chat with Alexey off list. Since 7162111 is indeed required
to get these tests running on headless systems we've agreed to go
ahead with this fix. (but not 8004928) A webrev with this change is
at:
http://cr.openjdk.java.net/~robm/7162111/webrev.02/
<http://cr.openjdk.java.net/%7Erobm/7162111/webrev.02/>
-Rob
On 20/12/12 22:11, Stuart Marks wrote:
On 12/19/12 12:32 AM, Alan Bateman wrote:
On 17/12/2012 18:46, Rob McKenna wrote:
Hi folks,
This review contains:
8003898: X11 toolkit can be chosen as the default toolkit
7162111: TEST_BUG: change tests run in headless mode [macosx]
(open)
8004928: TEST_BUG: Reduce dependence of CoreLib tests from the
AWT subsystem
Unfortunately the last two patches didn't apply cleanly, hence
the review
request. (the commit comments will be altered appropriately
before integration)
Webrev at:
http://cr.openjdk.java.net/~robm/7162111/webrev.01/
<http://cr.openjdk.java.net/%7Erobm/7162111/webrev.01/>
8003898 is important to get into jdk7u, but I don't think 7162111
or 8004928 is
really needed there.
Isn't 7162111 important to avoid hangs/failures when running the
tests on the Mac? Plus it removes tests from the problem list so
we get better test coverage in 7u.
s'marks