On 02/07/2013 21:55, huizhe wang wrote:
Thanks Sean. I added noreg-external label. I have a sub-task to
automate the test. Once it's done, do I need additional approval to
check in the test?
Joe,
are you going to use the same bug ID (8016133) to push the testcase?
Since you're pushing to different repo it's best thing to do. No need
for approval request in this case. Make sure your have review approvals etc.
regards,
Sean.
-Joe
On 7/2/2013 1:31 AM, Seán Coffey wrote:
Approved for jdk7u40-dev.
Please add the relevant noreg-* label to bug report.
regards,
Sean.
On 02/07/2013 05:56, huizhe wang wrote:
Hello.
I'd like to request approval to backport the following fix to
jdk7u40. This is a quick fix on a regression caused by a previous
patch. It is a recovery of code that was accidentally removed.
The bug:
http://bugs.sun.com/view_bug.do?bug_id=8016133
Webrev:
http://cr.openjdk.java.net/~joehw/jdk8/8016133/webrev/
Review:
http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-June/017870.html
JDK8 Changeset:
http://hg.openjdk.java.net/jdk8/jdk8/jaxp/rev/09d55894844d
Thanks,
Joe
