Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security

2011-03-07 Thread Rama Pulavarthi
Thanks for the review. We had to make additional changes to run the tests. Hopefully this is the final webrev for this issue. Please review this new webrev which also has changes to jdk/test/Makefile to run the tests. http://cr.openjdk.java.net/~ramap/CR7020513-open-webrev/ thanks, Rama

Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security

2011-03-07 Thread Sean Mullan
I'll defer to Kelly on the Makefile changes :) --Sean On 03/07/2011 02:39 PM, Rama Pulavarthi wrote: Thanks for the review. We had to make additional changes to run the tests. Hopefully this is the final webrev for this issue. Please review this new webrev which also has changes to

Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security

2011-03-07 Thread Alan Bateman
Rama Pulavarthi wrote: Thanks for the review. We had to make additional changes to run the tests. Hopefully this is the final webrev for this issue. Please review this new webrev which also has changes to jdk/test/Makefile to run the tests.

Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security

2011-02-28 Thread Mandy Chung
Rama, Looks good to me. Are you planning to add a regression test? You had it in the webrev of an earlier version. Mandy On 02/25/11 10:12, Rama Pulavarthi wrote: Please review this updated webrev that has the patch for JDK 7 repo.

Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security

2011-02-25 Thread Rama Pulavarthi
Please review this updated webrev that has the patch for JDK 7 repo. http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-tl-jaxws-7020513-open/webrev/ For background on this issue, this is not a new one. I am trying to port the old fixes made in jdk repo as part of earlier jax-ws integrations in to