From: "Eric Raible" <raible@gmail.com>
To: "Junio C Hamano" <gitster@pobox.com>
Cc: git@vger.kernel.org, Johannes.Schindelin@gmx.de
Subject: PATCH: allow ':/<oneline prefix>' notation to specify a specific file
Date: Thu, 3 Jul 2008 01:52:45 -0700 [thread overview]
Message-ID: <279b37b20807030152g13492d5dxf21367ab17719993@mail.gmail.com> (raw)
Although the rev-parse documentation claims that the tree-ish:path/to/file
syntax is applicable to all tree-ish forms this is not so when using the
:/ "oneline prefix" syntax introduced in v1.5.0.1-227-g28a4d94.
This patch allows git show ":/PATCH: allow":sha1_name.c to show the
change to the file changed by this patch.
Signed-off-by: Eric Raible <raible@gmail.com>
---
sha1_name.c | 15 +++++++++++++--
1 files changed, 13 insertions(+), 2 deletions(-)
diff --git a/sha1_name.c b/sha1_name.c
index b0b2167..a1acfcd 100644
--- a/sha1_name.c
+++ b/sha1_name.c
@@ -684,6 +684,7 @@ int get_sha1_with_mode(const char *name, unsigned
char *sha1, unsigned *mode)
int ret, bracket_depth;
int namelen = strlen(name);
const char *cp;
+ char *copy, *colon;
*mode = S_IFINVALID;
ret = get_sha1_1(name, namelen, sha1);
@@ -697,8 +698,18 @@ int get_sha1_with_mode(const char *name, unsigned
char *sha1, unsigned *mode)
int stage = 0;
struct cache_entry *ce;
int pos;
- if (namelen > 2 && name[1] == '/')
- return get_sha1_oneline(name + 2, sha1);
+ if (namelen > 2 && name[1] == '/') {
+ name += 2;
+ colon = strrchr(name, ':');
+ if (!get_sha1_oneline(name, sha1) || !colon)
+ return 0;
+ copy = xstrdup(name);
+ *(colon = strrchr(copy, ':')) = '\0';
+ ret = get_sha1_oneline(copy, sha1) ||
+ get_tree_entry(sha1, colon+1, sha1, mode);
+ free(copy);
+ return ret;
+ }
if (namelen < 3 ||
name[2] != ':' ||
name[1] < '0' || '3' < name[1])
--
1.5.6.1.1356.g3be5f.dirty
next reply other threads:[~2008-07-03 12:52 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-03 8:52 Eric Raible [this message]
2008-07-03 9:20 ` PATCH: allow ':/<oneline prefix>' notation to specify a specific file Junio C Hamano
2008-07-03 12:55 ` Johannes Schindelin
2008-07-03 12:53 ` Johannes Schindelin
-- strict thread matches above, loose matches on Subject: below --
2008-07-03 11:05 PATCH: Allow " Eric Raible
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=279b37b20807030152g13492d5dxf21367ab17719993@mail.gmail.com \
--to=raible@gmail.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/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).