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

Fixes @prefix counting (in case of 0) & adds new cmd 'nssort' & minor things #23

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

hoijui
Copy link
Contributor

@hoijui hoijui commented Jul 13, 2024

No description provided.

@hoijui hoijui changed the title Fixes @prefix counting (in case of 0), plus minor things Fixes @prefix counting (in case of 0) & adds new cmd 'nssort' & minor things Jul 13, 2024
@seebi
Copy link
Owner

seebi commented Jul 17, 2024

Hi @elevont --- running your new command results in an error - can you fix this?

∴ rdf nssort
/Users/seebi/.local/bin/rdf: line 1048: prefixfile: unbound variable

@hoijui
Copy link
Contributor Author

hoijui commented Jul 18, 2024

thank you @seebi !
should be fixed now.
(I before only tested the commands in the shell directly, instead of within the script, sorry :/ )

@seebi
Copy link
Owner

seebi commented Jul 26, 2024

@hoijui Given this file

@prefix : <https://sebastian.tramp.name/>.
@prefix foaf: <http://xmlns.com/foaf/0.1/>.

: a foaf:Person .

@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#>.
@prefix owl: <http://www.w3.org/2002/07/owl#>.
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>.

I get this error:

∴ rdf nssort index.ttl
head: illegal line count -- -1

Any idea?

@hoijui
Copy link
Contributor Author

hoijui commented Jul 29, 2024

Thanks for the testing! :-)
Apparently I was using head in a way that works on Linux but not on Mac.
It should now be fixed.

This also fixes spooky empty lines missing before the first prefix.
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.

2 participants