RFR: 8174735 Update JAX-WS RI integration to latest version
Aleks Efimov
aleksej.efimov at oracle.com
Wed Feb 15 23:49:46 UTC 2017
Hi,
The new webrev with addressed comments was uploaded here:
http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8174735/01
Best,
Aleksej
On 15/02/17 15:42, Roman Grigoriadi wrote:
> Hi Mandy,
>
> On 02/14/2017 11:53 PM, Mandy Chung wrote:
>>> On Feb 14, 2017, at 4:00 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-8174735
>>> Webrev:
>>> http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8174735/00/
>>
>> jaxws/src/jdk.xml.ws/share/classes/com/sun/tools/internal/ws/wscompile/WsimportTool.java
>>
>>
>> - /** JAXWS module name. JAXWS dependency is mandatory in
>> generated Java module. */
>> - private static final String JAXWS_MODULE = "java.xml.ws";
>> + /** JAXB module name. JAXB dependency is mandatory in generated
>> Java module. */
>> + private static final String JAXWS_PACKAGE = "java.xml.ws”;
> this looks to be merge failure on our side, will fix it again.
>>
>> JAXWS_MODULE is the right name as we discussed in the last JAX-WS
>> integration to jdk9. This change should be reverted and the upstream
>> project should be fixed.
>>
>> + if ("-release".equals(opt) && 9 >=
>> getVersion(javacOptions.get(i + 1))) {
> thanks, will be fixed to --release
>>
>> javac supports `—-release` (double dashes, GNU long form style) but
>> not the single dash option. Is this new option in wsimport and wsgen
>> tools? It should probably be consistent with javac.
>>
>> You can run jdeps —-check java.base,java.xml option to double check
>> if any remaining qualified exports to these modules.
>>
>> Otherwise looks okay.
>>
>> Mandy
>>
>
More information about the core-libs-dev
mailing list