Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

afs
Copy link
Contributor

@afs afs commented Mar 4, 2022

No description provided.

@afs
Copy link
Contributor Author

afs commented Mar 4, 2022

The productions for PN_CHARS_U are different between Turtle and N-Triples - this is visible in blank node labels.

N-triples is supposed to be subset of Turtle - it appears that it is N-triples that is wrong.

Turtle:

[164s] 	PN_CHARS_U 	::=   PN_CHARS_BASE | '_'

N-Triples:

[158s] PN_CHARS_U        ::=  PN_CHARS_BASE | '_' | ':'

This PR adds tests to turtle/ and ntriples/ assuming a correction for N-Triples.

@afs
Copy link
Contributor Author

afs commented Mar 5, 2022

I'll add copies to NQ and TriG.

It is looking increasing certain that NT and NQ are wrong and the intention was for there not to be : in the blank node label itself.

Let's wait to see if any parsers systems respond.

@lisp
Copy link

lisp commented Mar 5, 2022

Let's wait to see if any parsers systems respond.

we accept it as n-quads, as that is what the syntax says.
we do not accept it as sparql, as that is what the syntax says.

@Tpt
Copy link
Contributor

Tpt commented Mar 5, 2022

The "Rio" Rust implementation passes all these tests without change (we applied the Turtle version of the rule).

@gkellogg gkellogg requested a review from lisp April 25, 2022 21:25
@afs
Copy link
Contributor Author

afs commented May 15, 2022

Any reason not to merge this?

cc @lisp

@lisp
Copy link

lisp commented May 15, 2022

Any reason not to merge this?

in itself no.
i do admit, however, that we continue not to recognize this as a failure.
i note also, that the tool to which we refer clients as a neutral reference for syntax issues - rapper, is also not sensitive to the restriction.

@afs afs self-assigned this May 15, 2022
@gkellogg
Copy link
Member

@afs, I'd say merge at your discretion.

@afs
Copy link
Contributor Author

afs commented May 15, 2022

See also https://www.w3.org/2001/sw/wiki/RDF1.1_Errata#erratum_30 with email discussions link.

@afs afs merged commit d2cc355 into w3c:main May 15, 2022
@afs afs deleted the nt-bnode branch May 15, 2022 19:13
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.

4 participants