-
Notifications
You must be signed in to change notification settings - Fork 11
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
list without json format #35
Comments
Why not just use jq?
|
That's a option, but do you think that's a convenient option? I think clap-info should provide it. |
well. yes. the idea we had with clap-info was to produce a variety of structured data you could read with tools! If you want to add a
for instance extracts all the ids from a scan I don't think we want to add 100 options to take all the permutations of data when we can just output the data and use a widely available tool to parse it. |
oooh neat like here's another fun one
|
Who says 100 options? This is a request for one default/ reasonable option. |
yeah so i get what your request is. For one and only one case don't output json so you don't have to pipe and if anyone asks for any other case to do the same, say no, and tell them to use jq. Like I said if you want to add |
Anyway if you want to do that then clap-info/src/clap-info/main.cpp Line 148 in 271a6f3
that's where we make the list json. Add another option above for list raw, copy that branch to instead of appending to doc, dump to stdout, and then suppress the output of the document on return using the mechanism in the error paths elsewhere |
oh finally, clap-info is written so the scanner is a separate library from the executable itself, so you can use the C++ software to write other scan and inspect tools of your choosing! |
Comparing with lv2ls, lv2ls gives just a list of LV2 plugins/urls, while clap-info -l dumps it to stdout in json format. I think the lv2ls is useful on Unix systems, while it's easier to pipe with Unix tools like grep and such.
Clap-info could provide both ways probably.
The text was updated successfully, but these errors were encountered: