From: Linus Torvalds <torvalds@osdl.org>
To: Junio C Hamano <junkio@cox.net>
Cc: Carl Worth <cworth@cworth.org>, git@vger.kernel.org
Subject: Re: Two crazy proposals for changing git's diff commands
Date: Wed, 8 Feb 2006 17:30:56 -0800 (PST) [thread overview]
Message-ID: <Pine.LNX.4.64.0602081726390.2458@g5.osdl.org> (raw)
In-Reply-To: <7vfymtl43b.fsf@assigned-by-dhcp.cox.net>
On Wed, 8 Feb 2006, Junio C Hamano wrote:
>
> So we could even implement that with
>
> $ git commit --preview [other flags] [paths...]
My argument for "git status" was that it very much _is_ about "what would
I commit". So I'd much rather extend on "git status" than add a
"--preview" flag to "git commit".
At least to me, "git status" is very much a "what is pending" kind of
command. The fact that you can't con it into giving a diff is actually a
downside: I would at times have preferred to have the "git commit" message
contain an extended status that contained the diffs too.
Another way of saying that: I think it would make sense to have a verbose
status report, and maybe also have that verbose flag passed into "git
commit" so that you can see the verbose status when you edit the commit
message.
Under that logic, "git status -v" would show all the diffs (not just
filenames) and "git commit -v .." would be the same as "git commit .." but
the "-v" flag would have been passed down to the "git status" call, so the
commit message file would be pre-populated with the diff.
For small commits, it's actually nice to see the diff itself as you write
the commit message.
Linus
next prev parent reply other threads:[~2006-02-09 1:31 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-02-09 0:29 Two crazy proposals for changing git's diff commands Carl Worth
2006-02-09 1:05 ` Linus Torvalds
2006-02-09 1:21 ` Linus Torvalds
2006-02-09 23:07 ` Carl Worth
2006-02-09 23:40 ` Junio C Hamano
2006-02-09 1:35 ` Junio C Hamano
2006-02-09 1:21 ` Junio C Hamano
2006-02-09 1:30 ` Linus Torvalds [this message]
2006-02-09 1:37 ` Junio C Hamano
2006-02-10 9:05 ` Junio C Hamano
2006-02-10 20:32 ` Comments on "status -v" (was: Two crazy proposals for changing git's diff commands) Carl Worth
2006-02-10 21:09 ` Comments on "status -v" Junio C Hamano
2006-02-10 21:35 ` Linus Torvalds
2006-02-10 22:12 ` Junio C Hamano
2006-02-10 22:51 ` Petr Baudis
2006-02-10 23:26 ` Junio C Hamano
2006-02-09 23:44 ` Two crazy proposals for changing git's diff commands Carl Worth
2006-02-10 0:13 ` Junio C Hamano
2006-02-10 1:24 ` Carl Worth
2006-02-10 2:24 ` Junio C Hamano
2006-02-10 3:18 ` Carl Worth
2006-02-10 17:06 ` Mark Wooding
2006-02-13 9:23 ` Catalin Marinas
2006-02-13 22:00 ` Prune-safe StGIT (was Re: Two crazy proposals for changing git's diff commands) Catalin Marinas
2006-02-10 19:36 ` Two crazy proposals for changing git's diff commands Kent Engstrom
2006-02-11 19:25 ` Junio C Hamano
2006-02-12 12:00 ` [PATCH] Add howto about separating topics kent
2006-02-12 3:15 ` Two crazy proposals for changing git's diff commands J. Bruce Fields
2006-02-12 3:48 ` Junio C Hamano
2006-02-09 16:44 ` Tim Larson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Pine.LNX.4.64.0602081726390.2458@g5.osdl.org \
--to=torvalds@osdl.org \
--cc=cworth@cworth.org \
--cc=git@vger.kernel.org \
--cc=junkio@cox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).