-
Notifications
You must be signed in to change notification settings - Fork 5
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
TEST-XSPEC.xpl raises error in XML Calabash #22
Comments
There is a bug in MorganaXProc's static analysis: A missing connection for a (primary) output port is not correctly identified during static analysis. There should be an XS006, as correctly raised by XML Calabash 3. |
@xml-project yes it helps! thanks very much. In a rewrite we'll try to find a solution that works in both processors, or a stopgap which we can roll back when the Morgana bug fix is in. 🥬 |
@wendellpiez Any correct pipeline (accepted by XML Calabash) will also work with MorganaXProc-III (present and future). |
@xml-project thanks, that is helpful to know. |
Thanks, @xml-project and @wendellpiez ! |
Where the XProc has oscal-xproc3/xspec/xspec-execute.xpl Line 171 in bb76ea1
making it
Appears to satisfy XML Calabash. I.e. it is just a bit more scrupulous than Morgana. (Good lesson.) An alternative design could use |
You're right, only a non-functional XSpec would hit that. The XSpec compiler would raise an error if |
Let's close the Issue as Done For Now (we can revisit later if we like). |
The TEST-XSPEC.xpl pipeline works under Morgana, but raises an error under XML Calabash 3 alpha 9.
Side question about
TEST-XSPEC.xpl
and XML Calabash 3: Would having XML Calabash handle the.xspec
file name extension similar to.fo
as in this table be a reasonable enhancement request for XML Calabash? The first bullet of https://github.com/usnistgov/oscal-xproc3/blob/main/tutorial/sequence/Lesson02/walkthrough_102.md#test-xspec explains the workaround you had to do, and it seems that the default mappings in XML Calabash are trying to smooth that out. (Maybe you'd still need your workaround for Morgana, though.)The text was updated successfully, but these errors were encountered: