RFR: 8330542: Template for Creating Strict JAXP Configuration File [v13]

Alan Bateman alanb at openjdk.org
Fri May 24 11:46:07 UTC 2024


On Fri, 24 May 2024 05:26:40 GMT, Joe Wang <joehw at openjdk.org> wrote:

>> Add two sample configuration files:
>> 
>>   jaxp-strict.properties: used to set strict configuration, stricter than jaxp.properties in previous versions such as JDK 22
>> 
>>>   jaxp-compat.properties: used to regain compatibility from any more restricted configuration than previous versions such as JDK 22
>> 
>> Updated on 5/16/2024
>> 
>> Design change:
>> The design is changed to include in the JDK two configuration files that are the default jaxp.properties and jaxp-strict.properties, instead of three, dropping jaxp-compat.properties.
>> 
>> Updated on 5/18/2024
>> 
>> Withdraw changes to jaxp.properties. The original idea was to match jaxp-strict.properties with regard to the properties. However, that change impact the configuration process, resulting in tests that verify the process to fail.
>> 
>> Updated on 5/23/2024
>> 
>> Provide a template `jaxp-strict.template` instead of a properties file. This template can be used to create custom configuration files.
>
> Joe Wang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   add a template instead of a property file; remove implNote; update test and make script accordingly.

Marked as reviewed by alanb (Reviewer).

Magnus's suggestion for the suffix to be .properties.template makes sense, consistent with the one .template that the JDK currently includes in the run-time image.

Overall this looks okay, I'm happy that the implNote update is removed from the proposal as it read too much like the introducing a new supported interface and would have been confusing to have two configurations in the conf directory.

-------------

PR Review: https://git.openjdk.org/jdk/pull/18831#pullrequestreview-2076667358
PR Comment: https://git.openjdk.org/jdk/pull/18831#issuecomment-2129320078


More information about the build-dev mailing list