The fix is in the jdk repo, not in hotspot. Is it the repo or the Component in Jira that decides if approvals are needed?
/Staffan On 18 feb 2014, at 13:48, Seán Coffey <sean.cof...@oracle.com> wrote: > Results look good Dmitry. Since this is in hotspot category, you can push to > the hsx team forest. > You don't need approval on this mailing list. Alejandro will get fix into 7u > master via the bulk push request. > > regards, > Sean. > > On 18/02/2014 09:49, Dmitry Samersoff wrote: >> Sean, >> >> OK. Thank you for doing it. >> >> -Dmitry. >> >> On 2014-02-18 13:47, Seán Coffey wrote: >>> Dmitry, >>> >>> I've seen this type of testcase fail alot. I'm going to pull down your >>> patch and run a JPRT job on it. Will get back to you. >>> >>> regards, >>> Sean. >>> >>> On 18/02/2014 09:43, Dmitry Samersoff wrote: >>>> Seán, >>>> >>>> Updated CR with 8-na, updated webrev (in-place, press shift-reload) with >>>> changes to ProblemList.txt >>>> >>>> -Dmitry >>>> >>>> >>>> On 2014-02-18 13:19, Seán Coffey wrote: >>>>> Dmitry, >>>>> >>>>> Is this issue applicable for JDK 8u and JDK 9 ? There's no 8-na label on >>>>> the bug report. >>>>> Also - the testcase is listed in ProblemList.txt and hence doesn't run >>>>> through JPRT. >>>>> >>>>> Are you going to remove it from ProblemList if it's now working ? (i.e. >>>>> all JPRT runs pass) >>>>> >>>>> regards, >>>>> Sean. >>>>> >>>>> On 18/02/2014 07:54, Dmitry Samersoff wrote: >>>>>> Please approve. >>>>>> >>>>>> On 2014-02-14 22:21, Dmitry Samersoff wrote: >>>>>>> Hi Everyone, >>>>>>> >>>>>>> Please review the fix. >>>>>>> >>>>>>> The fix remove a colon (:) from file with reference output. >>>>>>> >>>>>>> http://cr.openjdk.java.net/~dsamersoff/JDK-8019800/webrev.01/ >>>>>>> >>>>>>> -Dmitry >>>>>>> >> >