drobilla / serd

A lightweight C library for RDF syntax
https://gitlab.com/drobilla/serd
ISC License
86 stars 15 forks source link

Does serdi support named pipe input/output ? #38

Closed davidandreoletti closed 1 year ago

davidandreoletti commented 1 year ago

Sample bash program:

mkfifo in;
mkfifo ou;
(serdi -i "Nquads" -o "NTriples" in > out) &;
write_to in &; // write_to is any program capable of writing to a named pipe
read_from out &; // read_form is any program capable of reading from a named pipe
drobilla commented 1 year ago

Please stop abusing issues to ask seemingly random questions you can answer yourself. Issues are for, well, actual issues.

davidandreoletti commented 1 year ago

@drobilla You should open Github Discussion (discussion tab), otherwise, I see no other alternative to reach you out for quick questions that might benefit other users. I agree this kind of question should not be raised in the Issues section.

Some other suggestions:

PS: The answer to the ticket's question is yes for everyone else.

drobilla commented 1 year ago

No offense, but I'm not actually interested in lazy drive-by questions. I do my best to support people who have done their due diligence and encounter an issue, but that's all. I appreciate real feedback, even on superficial things, but I do not appreciate armchair stuff that acts like my time has no value.

Updating serdi man page / README with other examples (named pipe, standard input, etc) would be useful. There is no mention of being able to read from standard input for example.

AFAIK there is nothing particularly special about named pipes. serdi will read from and write to more or less whatever, like most UNIX-style utilities, because that's a pretty fundamental concept of UNIX command lines. The help output says "Use - for INPUT to read from standard input", and besides, this too is a widespread convention. I have a hard time imagining anyone actually trying to do something with the software wouldn't at least try that, even if they didn't notice the help text.

This project has been around for more than a decade and it's never come up, so...

Redirecting users to some discord channel (if exists) or stackoverflow tag to check if the question is was asked.

It doesn't.

List of https://github.com/drobilla/serd/issues/37

People running some distro or package system don't generally look to project homepages to find out if a package exists. This is just another thing I would have to maintain which adds almost no value. Every packaging system I'm aware of has facilities for this, which are inherently correct and up-to-date. Most distros even have online ones.

davidandreoletti commented 1 year ago

The help output says "Use - for INPUT to read from standard input",

I did not find it in Archlinux v0.30.16-1 generated on Jul 15 2022.

I do not appreciate armchair stuff that acts like my time has no value.

Your time has value, as much as mine. We could have saved a little of each other time updating the doc. I was willing to provide a PR but given how the discussion has turned out, I pretty certain you would not accept the PR.

That being said, serdi is excellent. I thank you for it - genuinely.

drobilla commented 1 year ago

Your time has value, as much as mine. We could have saved a little of each other time updating the doc. I was willing to provide a PR but given how the discussion has turned out, I pretty certain you would not accept the PR.

This ticket was a question you could have - and apparently later did - easily answer yourself. It is your second such ticket on this project. Now, you've piled in a bunch of other things in the comments, and proceeded to use my response to this pattern of time wasting tickets as an argument for why I won't accept PRs for some completely unrelated thing that hadn't even been mentioned when I checked your behaviour?

Yeah... no, that's not how any of this works. Do better. The first time around, all you got was ellipses and a question mark. Now... well, here we are. There isn't a third level: zero tolerance instaban from here on out.