diff mbox

typo in btrfs-progs master/v3.16.x

Message ID AD5C5B4A-E27D-4206-8552-5483EDB1BE98@colorremedies.com (mailing list archive)
State Not Applicable
Headers show

Commit Message

Chris Murphy Aug. 25, 2014, 10:09 p.m. UTC 
git diff mason/master sterba/integration-20140729

Chris Murphy

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

David Sterba Aug. 25, 2014, 10:32 p.m. UTC | #1 
On Mon, Aug 25, 2014 at 04:09:16PM -0600, Chris Murphy wrote:
>  static const char * const cmd_scrub_start_usage[] = {
>         "btrfs scrub start [-BdqrRf] [-c ioprio_class -n ioprio_classdata] <path>|<device>",
> -       "Start a new scrub. If a scrub is already running, the new one fails.",
> +       "Start a new scrub",
>         "",
>         "-B     do not background",
>         "-d     stats per device (-B only)",
>         "-q     be quiet",
>         "-r     read only mode",
> -       "-R     raw print mode, print full data instead of summary",
> +       "-R     raw print mode, print full data instead of summary"
> 
> 
> Looks like a missing , at the end of this line. All other lines end in ,

Thanks for checking, the v3.16.x version is correct.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Chris Murphy Aug. 25, 2014, 10:43 p.m. UTC | #2 
On Aug 25, 2014, at 4:32 PM, David Sterba <dsterba@suse.cz> wrote:

> On Mon, Aug 25, 2014 at 04:09:16PM -0600, Chris Murphy wrote:
>> static const char * const cmd_scrub_start_usage[] = {
>>        "btrfs scrub start [-BdqrRf] [-c ioprio_class -n ioprio_classdata] <path>|<device>",
>> -       "Start a new scrub. If a scrub is already running, the new one fails.",
>> +       "Start a new scrub",
>>        "",
>>        "-B     do not background",
>>        "-d     stats per device (-B only)",
>>        "-q     be quiet",
>>        "-r     read only mode",
>> -       "-R     raw print mode, print full data instead of summary",
>> +       "-R     raw print mode, print full data instead of summary"
>> 
>> 
>> Looks like a missing , at the end of this line. All other lines end in ,
> 
> Thanks for checking, the v3.16.x version is correct.

Right. I had the diffs reversed from what I thought they were, so I came to the wrong conclusion.

Chris Murphy--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/cmds-scrub.c b/cmds-scrub.c
index 731c5c9..0bf06ee 100644
--- a/cmds-scrub.c
+++ b/cmds-scrub.c
@@ -1527,16 +1527,16 @@  out:
 
 static const char * const cmd_scrub_start_usage[] = {
        "btrfs scrub start [-BdqrRf] [-c ioprio_class -n ioprio_classdata] <path>|<device>",
-       "Start a new scrub. If a scrub is already running, the new one fails.",
+       "Start a new scrub",
        "",
        "-B     do not background",
        "-d     stats per device (-B only)",
        "-q     be quiet",
        "-r     read only mode",
-       "-R     raw print mode, print full data instead of summary",
+       "-R     raw print mode, print full data instead of summary"


Looks like a missing , at the end of this line. All other lines end in ,