External/cloud-hypervisor
1
0
Fork 0
mirror of https://github.com/cloud-hypervisor/cloud-hypervisor.git synced 2025-01-03 03:15:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

vhost_user_*: mark --*-backend as required

Browse Source 
I don't think the min_values(1) was doing anything, since
takes_value(true) was already set, and after this change I still get
an error if I try to supply the argument with no value.

But the unwrap below would fail if the argument wasn't supplied at
all.  By setting required(true) we get a nice message from clap
instead.

Signed-off-by: Alyssa Ross <hi@alyssa.is>
This commit is contained in:
Alyssa Ross 2021-08-04 14:36:47 +00:00 committed by Sebastien Boeuf
parent 3dff598fa2
commit deb4ad5e23
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
vhost_user_block/src/main.rs
View File

@ -25,7 +25,7 @@ fn main() {
.long("block-backend")
.help(vhost_user_block::SYNTAX)
.takes_value(true)
.min_values(1),
.required(true),
)
.get_matches();

2
vhost_user_net/src/main.rs
View File

@ -22,7 +22,7 @@ fn main() {
.long("net-backend")
.help(vhost_user_net::SYNTAX)
.takes_value(true)
.min_values(1),
.required(true),
)
.get_matches();