Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature] Allow usage of Saxon PE/EE #4813

Closed
kosek opened this issue Mar 20, 2023 · 1 comment · Fixed by #4854
Closed

[feature] Allow usage of Saxon PE/EE #4813

kosek opened this issue Mar 20, 2023 · 1 comment · Fixed by #4854
Labels
enhancement new features, suggestions, etc. regression
Milestone

Comments

@kosek
Copy link

kosek commented Mar 20, 2023

It should be possible to use licensed Saxon PE/EE as a replacement for bundled Saxon HE. In order to do that exist-db should provide ability to specify license key and configuration for Saxon PE/EE.

See comment from @adamretter

It looks like since eXist-db 6.1.0, it is not possible to configure it manually for Saxon-EE.
Basically eXist-db 6.1.0 was updated to use Saxon’s S9 API instead of the JAXP API. This made it possible for us to implement fn:transform#1 from XQuery 3.1 in a cleaner manner.
In fact I found a TODO in the code about this here - https://github.com/eXist-db/exist/blob/develop/exist-core/src/main/java/org/exist/xquery/functions/fn/transform/Transform.java#L88
So I am afraid you need to use an older version of eXist-db (I would not really recommend that), or yourself or someone else needs to make some changes to eXist-db to support Saxon-EE custom config file and license. I think the changes required to eXist-db should be relatively small and easy to achieve, but I am afraid I don’t have the time right now to delve into that.

@kosek kosek added the enhancement new features, suggestions, etc. label Mar 20, 2023
@adamretter adamretter added this to the eXist-6.2.1 milestone Mar 20, 2023
@adamretter
Copy link
Member

Closed by #4854

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement new features, suggestions, etc. regression
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants