stevedlawrence commented on pull request #745:
URL: https://github.com/apache/daffodil/pull/745#issuecomment-1027933417


   It looks like Saxon was added for schematron validation in commit 
d65af5b22a742a09594f277e8154dd4078cf3459. So I guess the rest of Daffodil won't 
use this new xml resolver and shouldn't have any issues. And I guess since it 
uses schematron files, it doesn't need to know about our resolution logic?
   
   In fact, looking at our schematron implementation it looks like it has added 
custom logic to allow resolving to files on the classpath:
   
   
https://github.com/apache/daffodil/blob/main/daffodil-schematron/src/main/scala/org/apache/daffodil/validation/schematron/ClassPathUriResolver.scala
   
   I think schematron and saxon are just completely separate from the rest of 
Daffodil, and so this new xmlresolver should be fine.


-- 
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]


Reply via email to