Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-18 Thread Naoto Sato
The fix has been pushed to T/L repository. Naoto On 9/18/12 1:40 AM, Alan Bateman wrote: On 17/09/2012 22:53, Naoto Sato wrote: OK, I filed a CR 7198984 for this and intend to push Eric's fix as it is. Alan, will you be a reviewer for this? Naoto That's fine. -Alan

Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-18 Thread Alan Bateman
On 17/09/2012 22:53, Naoto Sato wrote: OK, I filed a CR 7198984 for this and intend to push Eric's fix as it is. Alan, will you be a reviewer for this? Naoto That's fine. -Alan

Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-17 Thread Naoto Sato
OK, I filed a CR 7198984 for this and intend to push Eric's fix as it is. Alan, will you be a reviewer for this? Naoto On 9/15/12 3:41 AM, Alan Bateman wrote: On 14/09/2012 17:38, Naoto Sato wrote: I questioned this because of the following comment line in ProblemList.txt # More than one la

Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-15 Thread Alan Bateman
On 14/09/2012 17:38, Naoto Sato wrote: I questioned this because of the following comment line in ProblemList.txt # More than one label is allowed but must be on the same line. Which suggests that we could express the line like, java/text/Bidi/Bug6665028.java solaris-all linux-all But I co

Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-14 Thread Naoto Sato
On 9/14/12 2:29 AM, Alan Bateman wrote: In any case, you were right to propose adding the test to the exclude list. I see Naoto questions "generic-all" and that is a good topic to discuss. In this case the issue seems to be specific to C2 on x86 but that is too fine grained for the platform match

Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-14 Thread Alan Bateman
On 14/09/2012 02:35, Eric Wang wrote: Hi Naoto, Thank you to be my sponsor. I have checked the nightly results, this error also occurred on Linux as well. http://aurora.ru.oracle.com/functional/faces/RunDetails.xhtml?names=98081.CORELIBS-JDK8-NIGHTLY-JTREG-16 These links are Oracle internal l

Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-13 Thread Naoto Sato
Yes, and not on Windows, Mac, nor 64 bit systems, right? Naoto On 9/13/12 6:35 PM, Eric Wang wrote: Hi Naoto, Thank you to be my sponsor. I have checked the nightly results, this error also occurred on Linux as well. http://aurora.ru.oracle.com/functional/faces/RunDetails.xhtml?names=98081.COR

Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-13 Thread Eric Wang
Hi Naoto, Thank you to be my sponsor. I have checked the nightly results, this error also occurred on Linux as well. http://aurora.ru.oracle.com/functional/faces/RunDetails.xhtml?names=98081.CORELIBS-JDK8-NIGHTLY-JTREG-16 Thanks, Eric On 2012/9/14 4:23, Naoto Sato wrote: Hi Eric, I can be yo

Re: [PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-13 Thread Naoto Sato
Hi Eric, I can be your sponsor, but have one question. Looking at the bug report, the bug is only reproducible on solaris/linux x86. Does it have to disable the test on all the platforms with "generic-all"? Naoto On 9/13/12 12:26 AM, Eric Wang wrote: Hi, We want to add the bug 7196199

[PATCH] Review Request: Add bug 7196199 into ProblemList

2012-09-13 Thread Eric Wang
Hi, We want to add the bug 7196199 into ProblemList.txt. Can you please be my sponsor to review the patch attached and push it into repo if it is OK? Thanks, Eric diff -r f8c1cf072ba6 test/ProblemList.txt --- a/test/ProblemList.txt Wed