Hi Omair and Kumar,
As to me, the fix and the test look good. A few suggestions:
1. Mac is also a supported platform now, so you need to add the "Darwin"
option to the "SunOS | Windows_* )" case in the .sh file.
2. If gcc is not present, this should be considered that the test is
passed. Systems used for running tests may lack compilers, and this is
perfectly OK.
Having said that, let's wait and see what test Kumar has to offer.
--
best regards,
Anthony
On 08/11/12 00:56, Omair Majid wrote:
Hi Anthony,
On 08/10/2012 08:29 AM, Anthony Petrov wrote:
Actually, if Omair has a good automatic jtreg test, we would be happy to
get it checked in with this fix. Could we see a webrev for the test please?
I have uploaded a new webrev that includes a test case too:
http://cr.openjdk.java.net/~omajid/webrevs/jawt-link-regression/01/
The bug number in the jtreg test is entirely made up. Any thoughts for a
better description or alternate locations for the test code?
I can only run the test case on Linux, so that's what it's for. It needs
gcc to build the native bits. The webrev is against jdk8/awt forest, if
it matters.
Thanks,
Omair
