{"thread":{"id":"29660","subject":"[PATCH] remote: align set-branches builtin usage and documentation","startedAt":"2012-02-18T11:17:47Z","lastAt":"2012-02-18T13:11:44Z","messageCount":2,"participants":["Philip Jägenstedt"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"184927","messageId":"1329563867-13283-1-git-send-email-philip@foolip.org","threadId":"29660","inReplyTo":null,"subject":"[PATCH] remote: align set-branches builtin usage and documentation","fromName":"Philip Jägenstedt","fromEmail":"philip.jagenstedt@gmail.com","sentAt":"2012-02-18T11:17:47Z","receivedAt":"2012-02-18T11:17:47Z","isPatch":true,"sender":{"key":"philip@foolip.org","avatar":"https://avatars.githubusercontent.com/u/498917?v=4"},"body":"The order of [--add] <name> <branch>... was inconsistent\nbetween builtin remote usage messages and git-remote.txt.\nAlign it closer to the order used in set-url.\n\nSigned-off-by: Philip Jägenstedt <philip@foolip.org>\n---\n Documentation/git-remote.txt |    2 +-\n builtin/remote.c             |    2 +-\n 2 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-remote.txt b/Documentation/git-remote.txt\nindex 5a8c506..d376d19 100644\n--- a/Documentation/git-remote.txt\n+++ b/Documentation/git-remote.txt\n@@ -14,7 +14,7 @@ SYNOPSIS\n 'git remote rename' <old> <new>\n 'git remote rm' <name>\n 'git remote set-head' <name> (-a | -d | <branch>)\n-'git remote set-branches' <name> [--add] <branch>...\n+'git remote set-branches' [--add] <name> <branch>...\n 'git remote set-url' [--push] <name> <newurl> [<oldurl>]\n 'git remote set-url --add' [--push] <name> <newurl>\n 'git remote set-url --delete' [--push] <name> <url>\ndiff --git a/builtin/remote.c b/builtin/remote.c\nindex f54a89a..fec92bc 100644\n--- a/builtin/remote.c\n+++ b/builtin/remote.c\n@@ -16,7 +16,7 @@ static const char * const builtin_remote_usage[] = {\n \t\"git remote [-v | --verbose] show [-n] <name>\",\n \t\"git remote prune [-n | --dry-run] <name>\",\n \t\"git remote [-v | --verbose] update [-p | --prune] [(<group> | <remote>)...]\",\n-\t\"git remote set-branches <name> [--add] <branch>...\",\n+\t\"git remote set-branches [--add] <name> <branch>...\",\n \t\"git remote set-url <name> <newurl> [<oldurl>]\",\n \t\"git remote set-url --add <name> <newurl>\",\n \t\"git remote set-url --delete <name> <url>\",\n-- \n1.7.9.1.246.g77c8c.dirty\n"},{"id":"184930","messageId":"CAKHWUkY2ZNniJmhheHdxj4dzBLtY-+Pn4teAm2WE+0eTXYMYvg@mail.gmail.com","threadId":"29660","inReplyTo":"1329563867-13283-1-git-send-email-philip@foolip.org","subject":"Re: [PATCH] remote: align set-branches builtin usage and documentation","fromName":"Philip Jägenstedt","fromEmail":"philip@foolip.org","sentAt":"2012-02-18T13:11:44Z","receivedAt":"2012-02-18T13:11:44Z","isPatch":true,"sender":{"key":"philip@foolip.org","avatar":"https://avatars.githubusercontent.com/u/498917?v=4"},"body":"On Sat, Feb 18, 2012 at 12:17, Philip Jägenstedt\n<philip.jagenstedt@gmail.com> wrote:\n\n> The order of [--add] <name> <branch>... was inconsistent\n> between builtin remote usage messages and git-remote.txt.\n\nTo clarify, it's the order of \"git remote set-branches\" that is\ninconsistent with the two instances the patch touches:\n\nusage: git remote set-branches <name> <branch>...\n   or: git remote set-branches --add <name> <branch>...\n\n-- \nPhilip Jägenstedt\n"}]}