# [PATCH] Make git status usage say git status instead of git commit 6 messages from 2007-12-03 to 2007-12-03. Participants: Shawn Bohrer, Junio C Hamano, David Kastrup, Miklos Vajna, Florian Weimer, Eyvind Bernhardsen. Thread: https://gitlist.dev/t/11110 ## Shawn Bohrer, 2007-12-03 05:02 Subject: [PATCH] Make git status usage say git status instead of git commit Message-ID: <1196658129-16708-1-git-send-email-shawn.bohrer@gmail.com> URL: https://gitlist.dev/e/1196658129-16708-1-git-send-email-shawn.bohrer%40gmail.com ``` git status shares the same usage information as git commit since it shows what would be committed if the same options are given. However, when displaying the usage information for git status it should say it is for git status not git commit. Signed-off-by: Shawn Bohrer --- As a side question, should the usage information also use the non dash notation of the command since it is deprecated? I noticed all of the other commands are presently using the dash form, so I left it as is for now. builtin-commit.c | 15 ++++++++++----- 1 files changed, 10 insertions(+), 5 deletions(-) diff --git a/builtin-commit.c b/builtin-commit.c index f6e8e44..5e85a22 100644 --- a/builtin-commit.c +++ b/builtin-commit.c @@ -27,6 +27,11 @@ static const char * const builtin_commit_usage[] = { NULL }; +static const char * const builtin_status_usage[] = { + "git-status [options] [--] ...", + NULL +}; + static unsigned char head_sha1[20], merge_head_sha1[20]; static char *use_message_buffer; static const char commit_editmsg[] = "COMMIT_EDITMSG"; @@ -495,12 +500,12 @@ static void determine_author_info(struct strbuf *sb) strbuf_addf(sb, "author %s\n", fmt_ident(name, email, date, 1)); } -static int parse_and_validate_options(int argc, const char *argv[]) +static int parse_and_validate_options(int argc, const char *argv[], + const char * const usage[]) { int f = 0; - argc = parse_options(argc, argv, builtin_commit_options, - builtin_commit_usage, 0); + argc = parse_options(argc, argv, builtin_commit_options, usage, 0); if (logfile || message.len || use_message) no_edit = 1; @@ -597,7 +602,7 @@ int cmd_status(int argc, const char **argv, const char *prefix) git_config(git_status_config); - argc = parse_and_validate_options(argc, argv); + argc = parse_and_validate_options(argc, argv, builtin_status_usage); index_file = prepare_index(argc, argv, prefix); @@ -689,7 +694,7 @@ int cmd_commit(int argc, const char **argv, const char *prefix) git_config(git_commit_config); - argc = parse_and_validate_options(argc, argv); + argc = parse_and_validate_options(argc, argv, builtin_commit_usage); index_file = prepare_index(argc, argv, prefix); -- 1.5.3.6 ``` ## Junio C Hamano, 2007-12-03 05:34 Subject: Re: [PATCH] Make git status usage say git status instead of git commit Message-ID: <7vir3gqr3d.fsf@gitster.siamese.dyndns.org> URL: https://gitlist.dev/e/7vir3gqr3d.fsf%40gitster.siamese.dyndns.org In-Reply-To: <1196658129-16708-1-git-send-email-shawn.bohrer@gmail.com> ``` Shawn Bohrer writes: > git status shares the same usage information as git commit since it > shows what would be committed if the same options are given. However, > when displaying the usage information for git status it should say it > is for git status not git commit. > > Signed-off-by: Shawn Bohrer Thanks. Will apply. > As a side question, should the usage information also use the non dash > notation of the command since it is deprecated? I noticed all of the > other commands are presently using the dash form, so I left it as is for > now. Wise choice. We would probably want to clean them up at the same time early post 1.5.4. ``` ## David Kastrup, 2007-12-03 10:09 Subject: Re: [PATCH] Make git status usage say git status instead of git commit Message-ID: <854pf0hyy3.fsf@lola.goethe.zz> URL: https://gitlist.dev/e/854pf0hyy3.fsf%40lola.goethe.zz In-Reply-To: <7vir3gqr3d.fsf@gitster.siamese.dyndns.org> ``` Junio C Hamano writes: > Shawn Bohrer writes: > >> As a side question, should the usage information also use the non >> dash notation of the command since it is deprecated? I noticed all >> of the other commands are presently using the dash form, so I left it >> as is for now. > > Wise choice. We would probably want to clean them up at the same time > early post 1.5.4. Deprecating the dash form will probably mean that we need to think up (and document) a rationale for man git-commit I actually have typed something like man git commit a number of times already with obvious results. Finger memory. -- David Kastrup, Kriemhildstr. 15, 44793 Bochum ``` ## Florian Weimer, 2007-12-03 10:15 Subject: Re: [PATCH] Make git status usage say git status instead of git commit Message-ID: <877ijww0d3.fsf@mid.deneb.enyo.de> URL: https://gitlist.dev/e/877ijww0d3.fsf%40mid.deneb.enyo.de In-Reply-To: <854pf0hyy3.fsf@lola.goethe.zz> ``` * David Kastrup: > Deprecating the dash form will probably mean that we need to think up > (and document) a rationale for > > man git-commit > > I actually have typed something like > > man git commit > > a number of times already with obvious results. Finger memory. It's also a bit strange to have a git-commit(1) manpage when there is no git-commit on the path. ``` ## Miklos Vajna, 2007-12-03 10:21 Subject: Re: [PATCH] Make git status usage say git status instead of git commit Message-ID: <20071203102124.GY31750@genesis.frugalware.org> URL: https://gitlist.dev/e/20071203102124.GY31750%40genesis.frugalware.org In-Reply-To: <854pf0hyy3.fsf@lola.goethe.zz> ``` On Mon, Dec 03, 2007 at 11:09:24AM +0100, David Kastrup wrote: > Deprecating the dash form will probably mean that we need to think up > (and document) a rationale for > > man git-commit > > I actually have typed something like > > man git commit > > a number of times already with obvious results. Finger memory. that's why we have git help commit :) - VMiklos ``` ## Eyvind Bernhardsen, 2007-12-03 12:04 Subject: Re: [PATCH] Make git status usage say git status instead of git commit Message-ID: <116ABC64-B936-4EA3-9E8A-12FD71DDC1EA@orakel.ntnu.no> URL: https://gitlist.dev/e/116ABC64-B936-4EA3-9E8A-12FD71DDC1EA%40orakel.ntnu.no In-Reply-To: <877ijww0d3.fsf@mid.deneb.enyo.de> ``` On 3. des. 2007, at 11.15, Florian Weimer wrote: > * David Kastrup: > >> Deprecating the dash form will probably mean that we need to think up >> (and document) a rationale for >> >> man git-commit >> >> I actually have typed something like >> >> man git commit >> >> a number of times already with obvious results. Finger memory. > > It's also a bit strange to have a git-commit(1) manpage when there > is no > git-commit on the path. No stranger than having a perlfunc(1) manpage. bash-builtins is in section 7, maybe git-* should be moved there? Of course, then git wouldn't have any manpages in section 1 at all. Eyvind ```