Add basic CLI commands #1847

New Issue

Closed

opened 2025-10-09 17:32:51 +03:00 by OVERLORD · 10 comments

OVERLORD commented 2025-10-09 17:32:51 +03:00

Owner

Copy Link

Originally created by @xoxys on GitHub.

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. 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.

OVERLORD added the good first issue label 2025-10-09 17:32:51 +03:00

OVERLORD closed this issue 2025-10-09 17:32:53 +03:00

OVERLORD commented 2025-10-09 17:32:55 +03:00

Author

Owner

Copy Link

@tinywombat765 commented on GitHub:

I'd like to take this

@tinywombat765 commented on GitHub: I'd like to take this

OVERLORD commented 2025-10-09 17:32:56 +03:00

Author

Owner

Copy Link

@tinywombat765 commented on GitHub:

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: In that case I'd like to use structopt, which is built on top of clap but provides a nicer api imo

OVERLORD commented 2025-10-09 17:32:56 +03:00

Author

Owner

Copy Link

@tinywombat765 commented on GitHub:

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

@tinywombat765 commented on GitHub: @dani-garcia Do you have a preference as for as cli argument parsing is concerned?

OVERLORD commented 2025-10-09 17:32:56 +03:00

Author

Owner

Copy Link

@dani-garcia commented on GitHub:

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

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

OVERLORD commented 2025-10-09 17:32:56 +03:00

Author

Owner

Copy Link

@dani-garcia commented on GitHub:

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: 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

OVERLORD commented 2025-10-09 17:32:58 +03:00

Author

Owner

Copy Link

@dani-garcia commented on GitHub:

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: 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.

OVERLORD commented 2025-10-09 17:32:58 +03:00

Author

Owner

Copy Link

@tinywombat765 commented on GitHub:

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: 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?

OVERLORD commented 2025-10-09 17:32:58 +03:00

Author

Owner

Copy Link

@dani-garcia commented on GitHub:

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: 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) ```

OVERLORD commented 2025-10-09 17:32:59 +03:00

Author

Owner

Copy Link

@dani-garcia commented on GitHub:

Closed because #888 got merged

@dani-garcia commented on GitHub: Closed because #888 got merged

OVERLORD commented 2025-10-09 17:32:59 +03:00

Author

Owner

Copy Link

@tinywombat765 commented on GitHub:

I opened a PR #888

@tinywombat765 commented on GitHub: I opened a PR #888

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

main

test_dylint

1.34.3

1.34.2

1.34.1

1.34.0

1.33.2

1.33.1

1.33.0

1.32.7

1.32.6

1.32.5

1.32.4

1.32.3

1.32.2

1.32.1

1.32.0

1.31.0

1.30.5

1.30.4

1.30.3

1.30.2

1.30.1

1.30.0

1.29.2

1.29.1

1.29.0

1.28.1

1.28.0

1.27.0

1.26.0

1.25.2

1.25.1

1.25.0

1.24.0

1.23.1

1.23.0

1.22.2

1.22.1

1.22.0

1.21.0

1.20.0

1.19.0

1.18.0

1.17.0

1.16.3

1.16.2

1.16.1

1.16.0

1.15.1

1.15.0

1.14.2

1.14.1

1.14

1.13.1

1.13.0

1.12.0

1.11.0

1.10.0

1.9.1

1.9.0

1.8.0

1.7.0

1.6.1

1.6.0

1.5.0

1.4.0

1.3.0

1.2.0

1.1.0

1.0.0

0.13.0

0.12.0

0.11.0

0.10.0

0.9.0

Labels

Clear labels

better for forum

bug

bug

bug

documentation

duplicate

enhancement

future Vault

good first issue

help wanted

low priority

notes

pull-request

Mirrored from GitHub Pull Request

question

SSO

Third party

troubleshooting

troubleshooting

wontfix

No Label good first issue

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

OVERLORD

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/vaultwarden#1847