RFR: 8164479: Update JAX-WS RI integration to latest version

Alan Bateman Alan.Bateman at oracle.com
Sat Sep 24 09:53:47 UTC 2016


Joe - do you mind also checking the list of of qualified exports 
declared in java.xml's module-info as I assume that some of these should 
be removed as part of this patch (particularly 
com.sun.org.apache.xml.internal.resolver and resolver.tools).

-Alan


On 23/09/2016 21:29, Joe Wang wrote:
>
>
> On 9/23/16, 10:11 AM, Lance Andersen wrote:
>> Hi Roman,
>>
>> I made a pass through the webrev and overall, looks OK.  Would be 
>> good to have Joe Wang take a look at the Catalog changes as an 
>> additional sanity check.
>
> They look good to me. The dev note in Options.java,
>              // use Sun's "XML Entity and URI Resolvers" by Norman Walsh
>              // to resolve external entities.
> -            // http://www.sun.com/xml/developers/resolver/
> +            // 
> https://xerces.apache.org/xml-commons/components/resolver/resolver-article.html
>
> The whole comment can be changed to:
>
> // use javax.xml.catalog to resolve external entities.
>
> Best,
> Joe
>
>>
>> I am a bit concerned where we reference jaxb.java.net as in the 
>> package.html given we are not sure where the project will end up 
>> based on the pending changes coming to java.net.  For the 
>> specification, I would point to jcp.org vs java.net
>>
>> I am assuming the standalone TCKs in addition to the JCK  are clean 
>> when run against these changes?
>>
>> HTH
>>
>> Best
>> Lance
>>
>>> On Sep 23, 2016, at 6:39 AM, Roman 
>>> Grigoriadi<roman.grigoriadi at oracle.com>  wrote:
>>>
>>> Hi,
>>>
>>> Please review standalone JAXB/JAXWS changes, synced to jdk/jaxws repo.
>>>
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8164479
>>> Webrev: http://cr.openjdk.java.net/~aefimov/8164479/00/
>>>
>>> You can find change list in the description of JBS and its linked 
>>> issues. Most important is removal of code incompatible with JDK9 
>>> module system.
>>>
>>> With best regards,
>>> Roman
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif> 
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance 
>> Andersen| Principal Member of Technical Staff | +1.781.442.2037
>> Oracle Java Engineering
>> 1 Network Drive
>> Burlington, MA 01803
>> Lance.Andersen at oracle.com<mailto:Lance.Andersen at oracle.com>
>>
>>
>>



More information about the core-libs-dev mailing list