RFR: [jdk7u-dev] - 8003898: X11 toolkit can be chosen as the default toolkit

[Kelly O'Hair]

But the build uses applescript, preventing builds via ssh.

But I commend this effort, I consider it important that our testing is as robust as possible.

-kto
 
On Jan 25, 2013, at 7:37 AM, 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
>