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

psvi-required-001.xml should declare psvi-required #10

Closed
ndw opened this issue Aug 20, 2014 · 1 comment
Closed

psvi-required-001.xml should declare psvi-required #10

ndw opened this issue Aug 20, 2014 · 1 comment
Labels

Comments

@ndw
Copy link
Collaborator

ndw commented Aug 20, 2014

From private email:

While working with XProc test suite again, I am baffled by the
expected results of test "psvi-required-001.xml" over and over again.
Clearly an XProc implementation not supporting PSVI like Calumet or
mine cannot pass this test. But running the test it always
looks like I am producing wrong results on a supported step.

In my opinion reading the results of this test would be very much
clearer, if p:declare-step in the pipeline would use
attribute "psvi-required" so not supporting implementations can fail
with err:XD0022.

Would you agree or is there a reason to have the test in the current
version I did not get?

@ndw ndw added the bug label Aug 20, 2014
@ndw
Copy link
Collaborator Author

ndw commented Aug 20, 2014

Agreed in the 20 Aug 2014 XProc WG meeting.

ndw added a commit to ndw/test-suite that referenced this issue Aug 20, 2014
ndw added a commit that referenced this issue Aug 20, 2014
Resolve issue #10 by declaring psvi-required
@xproc xproc closed this as completed Aug 20, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant