Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security
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 Pulavarthi On 02/25/2011 12:21 PM, Sean Mullan wrote: Looks good to me. --Sean On 2/25/11 1:12 PM, Rama Pulavarthi wrote: 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 Open JDK 6. The original issue CR 6592792 was fixed in JDK 6u7 and then ported to Open JDK 6. When Open JDK 6 transitioned to Hg, it was committed as [1]. As fix for CR 6831313 http://monaco.sfbay.sun.com/detail.jsf?cr=6831313:update jaxws in OpenJDK7 to 2.1 plus bug fixes from OpenJDK 6, Only the changes in jaxws sources are ported to JDK 7 [2]. But, there are other changes that are made in jdk repo required for this fix. [1] http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/586feec8273d http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/586feec8273d. [2] http://hg.openjdk.java.net/jdk7/build/jaxws/rev/31822b475baa http://hg.openjdk.java.net/jdk7/build/jaxws/rev/31822b475baa. thanks, Rama Pulavarthi
Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security
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 jdk/test/Makefile to run the tests. http://cr.openjdk.java.net/~ramap/CR7020513-open-webrev/ thanks, Rama Pulavarthi On 02/25/2011 12:21 PM, Sean Mullan wrote: Looks good to me. --Sean On 2/25/11 1:12 PM, Rama Pulavarthi wrote: 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 Open JDK 6. The original issue CR 6592792 was fixed in JDK 6u7 and then ported to Open JDK 6. When Open JDK 6 transitioned to Hg, it was committed as [1]. As fix for CR 6831313 http://monaco.sfbay.sun.com/detail.jsf?cr=6831313:update jaxws in OpenJDK7 to 2.1 plus bug fixes from OpenJDK 6, Only the changes in jaxws sources are ported to JDK 7 [2]. But, there are other changes that are made in jdk repo required for this fix. [1] http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/586feec8273d http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/586feec8273d. [2] http://hg.openjdk.java.net/jdk7/build/jaxws/rev/31822b475baa http://hg.openjdk.java.net/jdk7/build/jaxws/rev/31822b475baa. thanks, Rama Pulavarthi
Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security
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. http://cr.openjdk.java.net/~ramap/CR7020513-open-webrev/ This looks good to me too. -Alan
Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security
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. 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 Open JDK 6. The original issue CR 6592792 was fixed in JDK 6u7 and then ported to Open JDK 6. When Open JDK 6 transitioned to Hg, it was committed as [1]. As fix for CR 6831313 http://monaco.sfbay.sun.com/detail.jsf?cr=6831313:update jaxws in OpenJDK7 to 2.1 plus bug fixes from OpenJDK 6, Only the changes in jaxws sources are ported to JDK 7 [2]. But, there are other changes that are made in jdk repo required for this fix. [1] http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/586feec8273d http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/586feec8273d. [2] http://hg.openjdk.java.net/jdk7/build/jaxws/rev/31822b475baa http://hg.openjdk.java.net/jdk7/build/jaxws/rev/31822b475baa. thanks, Rama Pulavarthi
Re: Review request for 7020513 : Add com.sun.xml.internal to the package.access property in java.security
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 Open JDK 6. The original issue CR 6592792 was fixed in JDK 6u7 and then ported to Open JDK 6. When Open JDK 6 transitioned to Hg, it was committed as [1]. As fix for CR 6831313 http://monaco.sfbay.sun.com/detail.jsf?cr=6831313:update jaxws in OpenJDK7 to 2.1 plus bug fixes from OpenJDK 6, Only the changes in jaxws sources are ported to JDK 7 [2]. But, there are other changes that are made in jdk repo required for this fix. [1] http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/586feec8273d http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/586feec8273d. [2] http://hg.openjdk.java.net/jdk7/build/jaxws/rev/31822b475baa http://hg.openjdk.java.net/jdk7/build/jaxws/rev/31822b475baa. thanks, Rama Pulavarthi