You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@daffodil.apache.org by GitBox <gi...@apache.org> on 2022/02/02 13:20:05 UTC

[GitHub] [daffodil] stevedlawrence commented on pull request #745: Update Saxon-HE to 11.1

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: commits-unsubscribe@daffodil.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org