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