[9] RFR: 8061466 - RELAX NG API visible but not accessible
Alan Bateman
Alan.Bateman at oracle.com
Fri Sep 25 11:04:34 UTC 2015
On 25/09/2015 11:46, Miroslav Kos wrote:
> :
>
> On 25/09/15 11:46, Alan Bateman wrote:
>>
>> On 25/09/2015 10:36, Miroslav Kos wrote:
>>> Hi,
>>> I prepared the webrev again with hg move:
>>> http://cr.openjdk.java.net/~mkos/8061466/jaxws.02/index.html
>>>
>>> For sure, jprt built again and all tests rerun.
>> The move and re-packaging looks good to me.
>>
>> However I see there is more to this as it drops the -relaxng and
>> -relaxng-compat command options so we'll need to make sure that this
>> is tracked (even though these command-line options seems to have
>> always been reported as unsupported and experimental).
> If it is necessary, I can file a CCC. Is it? If it is, I suppose it
> should address just change in tool (xjc) options - the implementation
> is in jdk.xml.bind, not java.xml.bind
>>
>> I see the localized versions of the properties have also been changed
>> and re-formatted. At one point then the rule was to not touch
>> localized files because they are generated/changed by translation
>> tools. I don't know know if the master version of these files is in
>> the jaxws repo or maintained somewhere upstream.
> The localized properties are in standalone repo, so I suppose it is
> master(?)
>
> Even if CCC required, it doesn't block us from pushing this, does it?
> If it was an issue, I can split the changeset into two (repackaging +
> removing options)
I think the main compatibility issue is that people may be relying on
these classes being present and not realizing it. That is, they might
have tools.jar but not relaxng.jar on the class path and will only
notice the issue when they try on JDK 9. So I think that point should be
tracked for the release notes. I agree that the command-line options
aren't too interesting but they can be mentioned too.
-Alan
More information about the core-libs-dev
mailing list