On Mon, 2020-02-10 at 11:13 +0800, Anand Jain wrote: > Grep failed to find this subcommand of btrfs, leading to a wrong > inference for a moment. > > Make sure we use the full subcommand name in the btrfs command. Well, I don't see how this could fail, but IMHO I like to see these commands written in full rather than using contracted names. > > Signed-off-by: Anand Jain <anand.jain@xxxxxxxxxx> Reviewed-by: Marcos Paulo de Souza <mpdesouza@xxxxxxxx> > --- > common/btrfs | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/common/btrfs b/common/btrfs > index 19ac7cc4b18c..33ad7e3b41cc 100644 > --- a/common/btrfs > +++ b/common/btrfs > @@ -7,7 +7,7 @@ _btrfs_get_subvolid() > mnt=$1 > name=$2 > > - $BTRFS_UTIL_PROG sub list $mnt | grep $name | awk '{ print $2 > }' > + $BTRFS_UTIL_PROG subvolume list $mnt | grep $name | awk '{ > print $2 }' > } > > # _require_btrfs_command <command> [<subcommand>|<option>]