{"thread":{"id":"39149","subject":"Re: [PATCH 1/5] status: document the -v/--verbose option","startedAt":"2015-04-24T00:27:47Z","lastAt":"2015-04-24T16:46:49Z","messageCount":3,"participants":["Pete Harlan","Junio C Hamano","Michael Haggerty"],"isPatch":true,"patchVersion":1,"patchTotal":5},"messages":[{"id":"259922","messageId":"CAM=ud8zLDTYUp12BsVjWH5=8rg=HxbtdBxL2L9x=3+FQVJBqrQ@mail.gmail.com","threadId":"39149","inReplyTo":null,"subject":"Re: [PATCH 1/5] status: document the -v/--verbose option","fromName":"Pete Harlan","fromEmail":"pchpublic88@gmail.com","sentAt":"2015-04-24T00:27:47Z","receivedAt":"2015-04-24T00:27:47Z","isPatch":true,"sender":{"key":"pchpublic88@gmail.com","avatar":null},"body":"Junio writes:\n> Michael Haggerty <mhagger@alum.mit.edu> writes:\n>\n> > Document `git status -v`, including its new doubled `-vv` form.\n> >\n> > Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>\n> >---\n>\n> Will queue on mg/status-v-v series, which did add description for\n> \"commit -v\", but \"status -v\" did not have the description to begin\n> with and we missed it.\n[...]\n> > +-v::\n> > +--verbose::\n> > + In addition to the names of files that have been changed, also\n> > + show the textual changes that are staged to be committed\n> > + (i.e., like the output of `git diff`). If `-v` is specified\n\nShould this be `git diff --cached`?\n\n> > + twice, then also show the changes in the working tree that\n> > + have not yet been staged (i.e., like the output of `git diff\n> > + --cached`).\n\n...and should this just be `git diff`?\n\n--Pete\n\n(Sorry for not replying to the email; bit of a mess here in my setup.)\n"},{"id":"259925","messageId":"xmqq4mo62tb9.fsf@gitster.dls.corp.google.com","threadId":"39149","inReplyTo":"CAM=ud8zLDTYUp12BsVjWH5=8rg=HxbtdBxL2L9x=3+FQVJBqrQ@mail.gmail.com","subject":"Re: [PATCH 1/5] status: document the -v/--verbose option","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2015-04-24T01:44:58Z","receivedAt":"2015-04-24T01:44:58Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Pete Harlan <pchpublic88@gmail.com> writes:\n\n> Junio writes:\n>> Michael Haggerty <mhagger@alum.mit.edu> writes:\n>>\n>> > Document `git status -v`, including its new doubled `-vv` form.\n>> >\n>> > Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>\n>> >---\n>>\n>> Will queue on mg/status-v-v series, which did add description for\n>> \"commit -v\", but \"status -v\" did not have the description to begin\n>> with and we missed it.\n> [...]\n>> > +-v::\n>> > +--verbose::\n>> > + In addition to the names of files that have been changed, also\n>> > + show the textual changes that are staged to be committed\n>> > + (i.e., like the output of `git diff`). If `-v` is specified\n>\n> Should this be `git diff --cached`?\n>\n>> > + twice, then also show the changes in the working tree that\n>> > + have not yet been staged (i.e., like the output of `git diff\n>> > + --cached`).\n>\n> ...and should this just be `git diff`?\n>\n> --Pete\n>\n> (Sorry for not replying to the email; bit of a mess here in my setup.)\n\nDuh!\n\nI see \"git commit\" documentation does not have these extra \"like the\noutput of...\" bits.  Perhaps we can do without it for the sake of\nboth brevity and risk avoidance?\n\nWill push out with --cached bit swapped for tonight on 'pu'.\n"},{"id":"259950","messageId":"553A7379.6030408@alum.mit.edu","threadId":"39149","inReplyTo":"CAM=ud8zLDTYUp12BsVjWH5=8rg=HxbtdBxL2L9x=3+FQVJBqrQ@mail.gmail.com","subject":"Re: [PATCH 1/5] status: document the -v/--verbose option","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2015-04-24T16:46:49Z","receivedAt":"2015-04-24T16:46:49Z","isPatch":true,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"On 04/24/2015 02:27 AM, Pete Harlan wrote:\n> Junio writes:\n>> Michael Haggerty <mhagger@alum.mit.edu> writes:\n>>\n>>> Document `git status -v`, including its new doubled `-vv` form.\n>>>\n>>> Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>\n>>> ---\n>>\n>> Will queue on mg/status-v-v series, which did add description for\n>> \"commit -v\", but \"status -v\" did not have the description to begin\n>> with and we missed it.\n> [...]\n>>> +-v::\n>>> +--verbose::\n>>> + In addition to the names of files that have been changed, also\n>>> + show the textual changes that are staged to be committed\n>>> + (i.e., like the output of `git diff`). If `-v` is specified\n> \n> Should this be `git diff --cached`?\n> \n>>> + twice, then also show the changes in the working tree that\n>>> + have not yet been staged (i.e., like the output of `git diff\n>>> + --cached`).\n> \n> ...and should this just be `git diff`?\n\nYes, you're right. Thanks for catching this, Pete. The shorter the\npatch, the higher the error density :-(\n\nMichael\n\n-- \nMichael Haggerty\nmhagger@alum.mit.edu\n"}]}