Just setup the object at construction time instead of changing it after
the fact, as we know off-hand what state it should be in. Use the new
constructor call form.
Change-Id: I5f7655eb083137b2af522038373cb7562fd7d14b
Make main() return a normal value, and let the call site act on it,
which in most cases implies doing an exit. This unifies the behavior
and gets rid of dead-code.
Change-Id: Id83284a95423b0d0e866d99f83bf546989054d76
There is no call that will pass such text as an argument, we only ever
pass the stringized list of missing parameters.
Change-Id: I1a9e2b05c59854c227685d53d9476c36086e38a9
The GetOptions() function will pass «$opt_name, $opt_value» to the
function refs in the options spec, which means that the usage()
function will consider that a missing parameters message and print
"help" as being a missing required parameter.
Use an anonymous function to wrap the usage() call and avoid the
unexpected arguments.
Change-Id: I5c10b241da23ccb7aa25a13dd5ebc1b4188b7370
We should only be printing relevant sections, not the entire man page,
which is rather verbose, and provides information not appropriate for a
--help output.
We use verbose level 1, for the commands that just have SYNOPSIS and
OPTIONS sections, and leave verbose level 99, but specify the relevant
sections which are 'SYNOPSIS|REQUIRED ARGUMENTS|OPTIONS'.
Change-Id: I59d1999599d553c396e60e183592c00e93d1a1db
We should not be outputting the usage() text when we get a run-time
error, as these are unrelated and depend on the state of the server,
not on how it was called.
Change-Id: I996d52cf2e4f4a834a281f8d85ea7571f61cf2e4
Names with underscores are more difficult to type. Use the prevalent
style of dashes as separators.
Add backwards compatibility symlinks to avoid breakage.
Change-Id: If6afcb1df4908ff6888afa3d9b2082eee4d3ed7d
Guillem Jover