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.

Sign up for GitHub

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

Jump to bottom

tests/manifest.ttl includes a non existing manifest #98

Closed
aucampia opened this issue May 10, 2022 · 1 comment · Fixed by #99
Closed

tests/manifest.ttl includes a non existing manifest #98

aucampia opened this issue May 10, 2022 · 1 comment · Fixed by #99

Comments

@aucampia
Copy link
Contributor

tests/manifest.ttl includes:

N3/tests/manifest.ttl

Lines 11 to 16 in c44d123

mf:include (
<TurtleTests/manifest.ttl>
<N3Tests/manifest.ttl>
<N3Tests/manifest-reasoner.ttl>
<N3Tests/manifest-extended.ttl>
) .

But there is no file N3Tests/manifest.ttl
@aucampia aucampia changed the title tests/manifest.ttl includes a non existing manifest: tests/manifest.ttl includes a non existing manifest May 10, 2022
@aucampia aucampia mentioned this issue May 10, 2022
Merged
@gkellogg
Copy link
Member

Yes, this looks like an error. PR #99 would seem to be the right fix.

aucampia added a commit to aucampia/rdflib that referenced this issue May 11, 2022
@aucampia
Revert to upstream manifest
92a0bf0 
Change `test/data/suites/w3c/n3/manifest.ttl` to the upstream version of
this file and workaround issues in it using in
 `test/test_w3c_spec/test_n3_w3c.py` by manually specyfing manifests.

The issue in the upstream manifest is tracked in:
- w3c/N3#98
aucampia added a commit to RDFLib/rdflib that referenced this issue May 12, 2022
@aucampia
Revert to upstream manifest
2ec532e 
Change `test/data/suites/w3c/n3/manifest.ttl` to the upstream version of
this file and workaround issues in it using in
 `test/test_w3c_spec/test_n3_w3c.py` by manually specyfing manifests.

The issue in the upstream manifest is tracked in:
- w3c/N3#98
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix tests/manifest.ttl aucampia/N3
2 participants
@gkellogg @aucampia