Replies: 4 comments
-
@udsamani I thought about this myself (#61), and I'm open to merging a PR to do so. qsv actually uses a fork of docopt with some clippy/perf tweaks, and for qsv's purposes - seems "good enough." I particularly like how the USAGE strings work, and how it's automagically parsed by docopt. But yeah - its unmaintained. Perhaps, we can leverage the same techniques cargo applied to migrate to clap in 2018 (rust-lang/cargo#5129)? Anyways, I'll reopen the ticket and put it in the backlog and would welcome PRs. |
Beta Was this translation helpful? Give feedback.
-
Now that clap v4.0 has been released, its time to revisit this issue. https://github.com/clap-rs/clap/blob/master/CHANGELOG.md#400---2022-09-28 |
Beta Was this translation helpful? Give feedback.
-
Still sticking with
Granted, |
Beta Was this translation helpful? Give feedback.
-
qsv now uses an optimized fork of docopt.rs - qsv_docoopt. As qsv has fully embraced docopt's self-documenting nature that neither clap nor structopt can provide, we'll maintain qsv_docopt going forward not only for general upkeep but to ensure it takes advantage of Rust's latest innovations & features. Besides, the two main known issues of docopt.rs (docopt/docopt.rs#207 and docopt/docopt.rs#152) are not critical to how we use it in qsv. For 207, its not relevant as qsv doesn't use docopt in that way. 152 is still valid though, and we've added it to qsv_docopt's backlog - jqnatividad/docopt.rs#2 |
Beta Was this translation helpful? Give feedback.
-
We are currently using 'docopt' for parsing arguments. It has following issues in my opinion
Beta Was this translation helpful? Give feedback.
All reactions