[RFC/PATCH 3/3] builtin/show.c: do not prune by pathspec
- From
Michael J Gruber <git@drmicha.warpmail.net>
- Date
- Mar 31, 2011, 09:17 UTC
- Message-ID
- <3bee7fb376e2fb498c9634ab2ff5506f8c74a7bc.1301562936.git.git@drmicha.warpmail.net>
- In-Reply-To
- <4D94322A.8030409@drmicha.warpmail.net>
By design, "git show commit -- path" is not "git show commit:path", and there is no reason to change that. But "git show commit -- path" simply returns nothing at all "most of the time" because it prunes by pathspec even though it does not walk commits. This is pretty useless.
So, turn off pruning (but keep diff limiting of course) so that "git show commit -- path" shows the commit message and the diff that the commit introduces to path (filtered by path); only the diff will be empty "most of the time".
As an intended side effect, users mistaking "git show commit -- path" for "git show commit:path" are automatically reminded that they asked git to show a commit, not a blob.
Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> --- builtin/log.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/builtin/log.c b/builtin/log.c index 9db43ed..ea0ddb4 100644 --- a/builtin/log.c +++ b/builtin/log.c @@ -411,6 +411,7 @@ int cmd_show(int argc, const char **argv, const char *prefix) opt.def = "HEAD"; opt.tweak = show_rev_tweak_rev; cmd_log_init(argc, argv, prefix, &rev, &opt); + rev.prune = 0; count = rev.pending.nr; objects = rev.pending.objects;
-- 1.7.4.2.668.gba03a4