Time to remove Xerces/Xalan dep from V3 POMs?
Ian Young
ian at iay.org.uk
Mon Jan 20 12:18:38 EST 2014
On 20 Jan 2014, at 16:42, Cantor, Scott <cantor.2 at osu.edu> wrote:
> I believe we've had prior discussions about intending to eliminate the
> requirement to endorse those jars and start supporting JAXP out of the box
> in the next version, but we haven't made the changes yet in the POMs. I
> think we should start testing and developing that way sooner rather than
> later.
+1
> I'm not sure if we need to explicitly reference the JAXP APIs the way we
> do for JSON and other things, since we're targeting Java 7 only, but I
> think we probably might want to to baseline the JAXP version we're
> depending on.
Per http://en.wikipedia.org/wiki/JAXP:
> J2SE versionJAXP version bundled
>
> 1.4 1.1
> 1.5 1.3
> 1.6 1.4
> 1.7.0 1.4.5
> 1.7.40 1.5
So, annoyingly, they switched JAXP version supported in the middle of Java 7, so there's a live issue as to whether we can assume JAXP 1.5 or not.
I experimented a bit with the MDA (because if I am using the old endorsed libraries I run into trouble with the new SchemaBuilder) and at least for the uses I have there, there is no need to include any dependencies explicitly.
I only ran up against two functional problems when I ripped those dependencies out:
* One place where some half-baked (now deleted) code was explicitly referencing an internal Xalan class
* A couple of places in tests where java-support's DomTypeSupport#longToDuration (to become DOMTypeSupport, I suppose) returned a more verbose but still valid result:
java.lang.AssertionError: expected [PT16M27.654S] but found [P0Y0M0DT0H16M27.654S]
So Xalan was apparently taking some care to suppress zero fields in Duration.toString() where the current Java 7 runtime is not. I should obviously make the test classes more flexible to allow for variation here.
I'm not by any means saying that's all we'll see, but it probably bodes well that I didn't end up with lots more.
-- Ian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5943 bytes
Desc: not available
Url : http://shibboleth.net/pipermail/dev/attachments/20140120/b001bfc2/attachment.bin
More information about the dev
mailing list