common: _require_command needs to handle parameters
authorDave Chinner <dchinner@redhat.com>
Mon, 4 May 2015 06:48:05 +0000 (16:48 +1000)
committerDave Chinner <david@fromorbit.com>
Mon, 4 May 2015 06:48:05 +0000 (16:48 +1000)
commit9435b928bf82c3ebefce667b512d14845e84b82f
treefb34292ed93bee83730ba9edc07f024feb133cfb
parentd5425f4db9cb66cd02b36da5336528426ff3ffc7
common: _require_command needs to handle parameters

_require_command fails when a parameter based command is passed to
it, such as "xfs_io -F" or "btrfs filesystem defrag" as the command
string does not point at a binary.  Rather than hacking at all the
callers and limiting what we can do with $*_PROGS variables, just
make _require_command handle this case sanely.

Change _require_command to check for one or two variables passed to
it and to fail if none or more than 2 parameters are passed. This
will catch most cases where unquoted parameter-based commands are
passed. Further, for the command variable, the executable we need to
check for is always going to be the first token in the variable.
Hence we can simply ignore everything after the first token for the
purposes of existence and executable checks on the command.

Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
common/rc