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

[rdf2hdt] -B flag doesn't seem to work #131

Open
sharpaper opened this issue Oct 30, 2017 · 0 comments
Open

[rdf2hdt] -B flag doesn't seem to work #131

sharpaper opened this issue Oct 30, 2017 · 0 comments
Labels
bug

Comments

@sharpaper
Copy link

When parsing a Turtle file with an empty object like this

<http://example.org/Mary>
    ex:name "Mary" ;
    ex:picture <> .

using the -B flag, rdf2hdt throws an error whereas it should use the basename (provided by -B) instead.
@sharpaper sharpaper mentioned this issue Oct 30, 2017
Closed
@wouterbeek wouterbeek mentioned this issue Jul 23, 2018
Closed
wouterbeek added a commit that referenced this issue Jul 24, 2018
@wouterbeek
ADDED: Support for setting a base URI.
a089d5e 
As observed in issue #131, setting the base URI did not work
previously.  As a result, the `-B' flag in the `rdf2hdt' script did
not work either.

This commit adds base URI support by using the Serd API to resolve
relative IRIs while parsing RDF input.

In the old code, base URIs were explicitly rewritten to use Turtle
notation (i.e., using surrounding angular brackets).  Since URI
resolution is now handled by the Serd API, such serialization-specific
reformats are no longer needed and have been removed from the code.

In line with the previous point, the `-B' flag now takes a base URI
argument directly, and does not require surrounding angular brackets.
wouterbeek added a commit that referenced this issue Jul 24, 2018
@wouterbeek
ADDED: Support for setting a base URI.
553d322 
As observed in issue #131, setting the base URI did not work
previously.  As a result, the `-B' flag in the `rdf2hdt' script did
not work either.

This commit adds base URI support by using the Serd API to resolve
relative IRIs while parsing RDF input.

In the old code, base URIs were explicitly rewritten to use Turtle
notation (i.e., using surrounding angular brackets).  Since URI
resolution is now handled by the Serd API, such serialization-specific
reformats are no longer needed and have been removed from the code.

In line with the previous point, the `-B' flag now takes a base URI
argument directly, and does not require surrounding angular brackets.
@wouterbeek wouterbeek mentioned this issue Jul 24, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@LaurensRietveld @sharpaper