Add basic CLI commands #4207

Closed
opened 2026-03-07 19:33:32 -06:00 by GiteaMirror · 10 comments
Owner

Originally created by @xoxys on GitHub (Jan 16, 2020).

It could be handy to have some basic CLI options for the bitwarden_rs binary in place. Minimum set could be something like bitwarden_rs --help and bitwarden_rs --version.

These CLI options can also be used to verify if the compiled binary is executable.

Originally created by @xoxys on GitHub (Jan 16, 2020). It could be handy to have some basic CLI options for the `bitwarden_rs` binary in place. Minimum set could be something like `bitwarden_rs --help` and `bitwarden_rs --version`. These CLI options can also be used to verify if the compiled binary is executable.
GiteaMirror added the good first issue label 2026-03-07 19:33:32 -06:00
Author
Owner

@tinywombat765 commented on GitHub (Mar 2, 2020):

I'd like to take this

@tinywombat765 commented on GitHub (Mar 2, 2020): I'd like to take this
Author
Owner

@dani-garcia commented on GitHub (Mar 2, 2020):

Sure thing, have at it! If you need help I'm usually either here or the Matrix chat.

@dani-garcia commented on GitHub (Mar 2, 2020): Sure thing, have at it! If you need help I'm usually either here or the Matrix chat.
Author
Owner

@tinywombat765 commented on GitHub (Mar 2, 2020):

@dani-garcia Do you have a preference as for as cli argument parsing is concerned?

@tinywombat765 commented on GitHub (Mar 2, 2020): @dani-garcia Do you have a preference as for as cli argument parsing is concerned?
Author
Owner

@dani-garcia commented on GitHub (Mar 2, 2020):

Do you mean choice of library? If nothing has changed, clap is the most popular so I'd go that way.

https://github.com/clap-rs/clap

@dani-garcia commented on GitHub (Mar 2, 2020): Do you mean choice of library? If nothing has changed, clap is the most popular so I'd go that way. https://github.com/clap-rs/clap
Author
Owner

@tinywombat765 commented on GitHub (Mar 2, 2020):

In that case I'd like to use structopt, which is built on top of clap but provides a nicer api imo

@tinywombat765 commented on GitHub (Mar 2, 2020): In that case I'd like to use structopt, which is built on top of clap but provides a nicer api imo
Author
Owner

@dani-garcia commented on GitHub (Mar 2, 2020):

Ah, I thought the structopt api was merged into clap too, but apparently that's scheduled to be released on the next version of clap 3.0!

I agree the declarative api of structopt is nicer, I'd prefer to use that too.

@dani-garcia commented on GitHub (Mar 2, 2020): Ah, I thought the structopt api was merged into clap too, but apparently that's scheduled to be released on the next version of clap 3.0! I agree the declarative api of structopt is nicer, I'd prefer to use that too.
Author
Owner

@tinywombat765 commented on GitHub (Mar 2, 2020):

I see that if the GIT_VERSION environment var is set at compile time the app with print that as it's version. Should I use that for the version flag? If so what should it print if anything it that var is not set? And if not what should I print?

@tinywombat765 commented on GitHub (Mar 2, 2020): I see that if the GIT_VERSION environment var is set at compile time the app with print that as it's version. Should I use that for the version flag? If so what should it print if anything it that var is not set? And if not what should I print?
Author
Owner

@dani-garcia commented on GitHub (Mar 2, 2020):

For the version flag I'd just print bitwarden_rs followed by a space and the GIT_VERSION value if present. Similar to how cargo --version or rustc --version do it.

If no version is provided, you could print something like.

bitwarden_rs (Version info from Git not present)
@dani-garcia commented on GitHub (Mar 2, 2020): For the version flag I'd just print `bitwarden_rs` followed by a space and the GIT_VERSION value if present. Similar to how cargo --version or rustc --version do it. If no version is provided, you could print something like. ``` bitwarden_rs (Version info from Git not present) ```
Author
Owner

@tinywombat765 commented on GitHub (Mar 2, 2020):

I opened a PR #888

@tinywombat765 commented on GitHub (Mar 2, 2020): I opened a PR #888
Author
Owner

@dani-garcia commented on GitHub (Mar 2, 2020):

Closed because #888 got merged

@dani-garcia commented on GitHub (Mar 2, 2020): Closed because #888 got merged
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#4207