Review Request[7] JMC-6283 Update Javamail api and its dependencies
Guru
guru.hb at oracle.com
Fri Mar 1 06:55:45 UTC 2019
+1
I have verified your fix applying to "http://hg.openjdk.java.net/jmc/jmc” rather than "http://hg.openjdk.java.net/jmc/jmc7” as you have created webrev based on jmc7 repo.
Please commit the changes to "http://hg.openjdk.java.net/jmc/jmc” repo and I will do the auto backport using hg merge.
Merging changes from jmc7 to jmc is bit clumsy , where as the other way is cleaner and simpler.
Thanks,
Guru
> On 01-Mar-2019, at 11:09 AM, Prem Balakrishnan <prem.balakrishnan at oracle.com> wrote:
>
> Hi All,
>
>
>
> Please review the patch for JMC7.0
>
> Bug: https://bugs.openjdk.java.net/browse/JMC-6283
>
> Webrev: http://cr.openjdk.java.net/~pkbalakr/jmc/6283/webrev.00/
>
>
>
> Earlier we had reverted the fix for bug https://bugs.openjdk.java.net/browse/JMC-6217 "Update mail.api and its dependencies".
>
> Now we have all the necessary third-party approvals, re-submitting the patch for review.
>
>
>
> Regards,
>
> Prem
More information about the jmc-dev
mailing list