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
