From 5ad7cce12a8af75447ce45311dc2ef9883116041 Mon Sep 17 00:00:00 2001 From: Matthias Beyer Date: Thu, 10 Nov 2022 16:01:15 +0100 Subject: [PATCH 1/2] Fix: Number of args for --env in "build" subcommand Fixes: a53f3bfd0622f1ee955af053c52f1d1577695c90 ("Rewrite value fetching") Signed-off-by: Matthias Beyer --- src/cli.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cli.rs b/src/cli.rs index d9b36f4a..512101d3 100644 --- a/src/cli.rs +++ b/src/cli.rs @@ -465,7 +465,7 @@ pub fn cli() -> Command { .arg(Arg::new("env") .required(false) - .num_args(0) + .num_args(1) .short('E') .long("env") .value_parser(ValueParser::new(env_pass_validator)) From 42b40b5d857c8dc3caa931546bd83c161d9372af Mon Sep 17 00:00:00 2001 From: Matthias Beyer Date: Sat, 12 Nov 2022 15:11:12 +0100 Subject: [PATCH 2/2] Fix: Specify the argument action Signed-off-by: Matthias Beyer --- src/cli.rs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/cli.rs b/src/cli.rs index 512101d3..ae695893 100644 --- a/src/cli.rs +++ b/src/cli.rs @@ -14,6 +14,7 @@ use clap::crate_authors; use clap::crate_version; use clap::Command; use clap::Arg; +use clap::ArgAction; use clap::ArgGroup; use clap::builder::PossibleValuesParser; use clap::builder::ValueParser; @@ -467,6 +468,7 @@ pub fn cli() -> Command { .required(false) .num_args(1) .short('E') + .action(ArgAction::Set) .long("env") .value_parser(ValueParser::new(env_pass_validator)) .help("Pass environment variable to all build jobs")