Btrfs-progs: update usage message for cmds-restore
authorFilipe David Borba Manana <fdmanana@gmail.com>
Wed, 10 Jul 2013 08:36:34 +0000 (09:36 +0100)
committerDavid Sterba <dsterba@suse.cz>
Fri, 9 Aug 2013 12:32:35 +0000 (14:32 +0200)
Mention that a target path argument is mandatory unless
the -l option is supplied. Also mention about the existence
of the -l option, which was previously not announced.

$ btrfs restore -v /dev/sdb3
usage: btrfs restore [options] <device>

    Try to restore files from a damaged filesystem (unmounted)

    -s              get snapshots
    -v              verbose
    -i              ignore errors
    -o              overwrite
    -t              tree location
    -f <offset>     filesystem location
    -u <block>      super mirror
    -d              find dir
$ echo $?
129

After specifying a target path, the command works as expected:

$ btrfs restore -v /dev/sdb3 files2/
Restoring files2/file1
Done searching
$ echo $?
0

V2: Updated command synopsis by suggestion of Anand Jain.

Signed-off-by: Filipe David Borba Manana <fdmanana@gmail.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
cmds-restore.c

index a9a8da7..712b672 100644 (file)
@@ -974,7 +974,7 @@ static struct option long_options[] = {
 };
 
 const char * const cmd_restore_usage[] = {
-       "btrfs restore [options] <device>",
+       "btrfs restore [options] <device> <path> | -l <device>",
        "Try to restore files from a damaged filesystem (unmounted)",
        "",
        "-s              get snapshots",
@@ -985,6 +985,7 @@ const char * const cmd_restore_usage[] = {
        "-f <offset>     filesystem location",
        "-u <block>      super mirror",
        "-d              find dir",
+       "-l              list roots",
        "--path-regex <regex>",
        "                restore only filenames matching regex,",
        "                you have to use following syntax (possibly quoted):",