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