ppkarwasz commented on PR #640: URL: https://github.com/apache/commons-configuration/pull/640#issuecomment-4740937637
> -1: We've already discussed making an new Commons component for this code, let's do that please. Sorry, I was busy iterating on code and I forgot to mention that I created this PR only to test the capabilities (or missing capabilities) of `copernik-xml-factory`, **before** it gets included as a separate Commons component. For a new component to be useful, it should be adopted by at least a couple of projects. Currently I have tested support for such a library: - In Log4j (apache/logging-log4j2#4144), where it appears that the library should have separate self-contained public classes, so that code necessary by a single JAXP implementation can be shaded. This does not exclude the presence of an `XmlFactories` common utility, - In Batik ([BATIK-1395](https://issues.apache.org/jira/browse/BATIK-1395)), where the idea of replacing stock JDK-specific hardenings with an external implementation-independent library didn't find good ground, - Calcite ([CALCITE-7609](https://issues.apache.org/jira/browse/CALCITE-7609)) is interested, so I will test how well adapted the library is to their code, - In this PR I am exploring the applicability of the library to Configuration. I already found one bug (copernik-eu/copernik-xml-factory#28) and a problem with JDK 8 and schema validation. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
