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

d1780014fa3a got xmldb:copy backwards #219

Closed
craigberry opened this issue Apr 19, 2019 · 1 comment · Fixed by #220
Closed

d1780014fa3a got xmldb:copy backwards #219

craigberry opened this issue Apr 19, 2019 · 1 comment · Fixed by #220
Labels

Comments

@craigberry
Copy link

When building the current develop-4.x.x branch of eXist and then trying to run it, eXide is non-functional. There are errors such as this:

err:XPST0017 Function xmldb:copy-resource() is not defined in module namespace: http://exist-db.org/xquery/xmldb [at line 223, column 13]

It seems that in replacing functions that are deprecated for 5.x, instances of xmldb:copy have been replaced with the deprecated equivalents:

$ grep -R 'xmldb:copy-' *
modules/collections.xql:                                xmldb:copy-collection($source, $target)
modules/collections.xql:                                xmldb:copy-resource($split[1], $split[2], $target, $split[2])
modules/deployment.xql:            xmldb:copy-resource($source, $resource, $target, $resource),

Apparently those functions are already gone in eXist 4.x, or so I surmise from the errors I get in:

$ git describe
eXist-4.6.1-120-g78a06bdcc
@craigberry
Copy link
Author

It may be me that has it backwards, but I'm confused by a couple of things. The release notes for eXist 5.0 RC1 say: "Implemented simplified xmldb:copy-resource and xmldb:copy-collection functions." But in the ticket I opened in the eXist repository about deprecations where @joewiz provided a list of what is being replaced with what, it clearly has the direction of deprecation going the other way, with these functions being replaced by xmldb:copy.

So maybe the real bug is that eXist 4.x is now broken with eXide 2.4.7 and its new 5.x-only functions and the build is not smart enough to pull in an older (compatible) version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant