Skip to content

add missing fn ns declaration#197

Merged
namedgraph merged 1 commit intoAtomGraph:masterfrom
MikiDi:patch-1
Nov 7, 2024
Merged

add missing fn ns declaration#197
namedgraph merged 1 commit intoAtomGraph:masterfrom
MikiDi:patch-1

Conversation

@MikiDi
Copy link
Copy Markdown
Contributor

@MikiDi MikiDi commented Nov 7, 2024

Hi!

Building the latest version, (aa6be41) through Docker fails as follows for me:

65.78 [INFO] --- frontend-maven-plugin:1.12.1:npx (generate-sef) @ linkeddatahub ---
65.78 [INFO] Running 'npx xslt3 -t -xsl:/usr/src/platform/target/ROOT/static/com/atomgraph/linkeddatahub/xsl/client.xsl -export:/usr/src/platform/target/ROOT/static/com/atomgraph/linkeddatahub/xsl/client.xsl.sef.json -nogo -ns:##html5 -relocate:on' in /usr/src/platform
66.31 [INFO] npm WARN exec The following package was not found and will be installed: xslt3@2.7.0
66.98 [INFO] SaxonJS 2.7 from Saxonica 
66.98 [INFO] Node.js version v20.8.1
66.98 [INFO] Compiling stylesheet /usr/src/platform/target/ROOT/static/com/atomgraph/linkeddatahub/xsl/client.xsl
82.69 [INFO] Error XPST0081:
82.69 [INFO]   Static error in XPath on line 48 {analyze-string($query-string, '[^a-zA-Z]?(SELECT|ASK|DESCRIBE|CONSTRUCT)[^a-zA-Z]', 'i')/fn:match[1]/fn:group[@nr = '1']/string()}: No namespace binding for prefix 'fn'
82.69 [INFO] Failed to compile stylesheet
82.77 [INFO] npm notice 
82.77 [INFO] npm notice New minor version of npm available! 10.1.0 -> 10.9.0
82.77 [INFO] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v10.9.0>
82.77 [INFO] npm notice Run `npm install -g npm@10.9.0` to update!
82.77 [INFO] npm notice 
82.79 [INFO] ------------------------------------------------------------------------
82.79 [INFO] BUILD FAILURE
82.79 [INFO] ------------------------------------------------------------------------

This change should fix that.

@namedgraph
Copy link
Copy Markdown
Member

Thanks @MikiDi. I noticed this on the develop branch when SaxonJS automatically upgraded to 2.7 (must have stricter namespace checking) but forgot to backport it.

@namedgraph namedgraph merged commit 0080cf4 into AtomGraph:master Nov 7, 2024
@namedgraph
Copy link
Copy Markdown
Member

Released 4.0.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants