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
