[libcamera-devel] [PATCH v2] cam: options: optional arguments needs to be specified as --foo=bar

Niklas Söderlund niklas.soderlund at ragnatech.se
Fri Jan 25 18:42:54 CET 2019


It's not state in the documentation but optional arguments needs to be
specified using as '--foo=bar' instead of '--foo bar', otherwise the
value is not propagated to optarg during argument parsing. Update the
usage printing helper to reflect this requirement.

Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
---
 src/cam/options.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/cam/options.cpp b/src/cam/options.cpp
index 82acff9bbeea476d..55c42540f92478e6 100644
--- a/src/cam/options.cpp
+++ b/src/cam/options.cpp
@@ -143,9 +143,10 @@ void OptionsParser::usage()
 		};
 
 		if (option.argument != ArgumentNone) {
-			argument += std::string(" ");
 			if (option.argument == ArgumentOptional)
-				argument += "[";
+				argument += "[=";
+			else
+				argument += " ";
 			argument += option.argumentName;
 			if (option.argument == ArgumentOptional)
 				argument += "]";
-- 
2.20.1



More information about the libcamera-devel mailing list