Rather than erroring out, treat an empty path as the path to
the root of a tree so frontends can be simplified a little.
While at it, fix a typo in an error message: the cat command is used
to examine paths within trees, not branches.
Cc: David Barr <david.barr@xxxxxxxxxxxx>
Signed-off-by: Jonathan Nieder <jrnieder@xxxxxxxxx>
---
Would this be helpful? I imagine it would make tree access by
pathname a bit simpler, and I found myself tempted to try it.
The test from the next patch exercises this.
fast-import.c | 6 +++++-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/fast-import.c b/fast-import.c
index 099f63e..f3c4123 100644
--- a/fast-import.c
+++ b/fast-import.c
@@ -2702,9 +2702,13 @@ static void quoted_path_sha1(unsigned char sha1[20], struct tree_entry *root,
die("Invalid path: %s", line);
if (*x)
die("Garbage after path: %s", line);
+ if (uq.len == 0) {
+ hashcpy(sha1, root->versions[1].sha1);
+ return;
+ }
tree_content_get(root, uq.buf, &leaf);
if (!leaf.versions[1].mode)
- die("Path %s not in branch", uq.buf);
+ die("Path %s not in tree", uq.buf);
hashcpy(sha1, leaf.versions[1].sha1);
}
--
1.7.3.rc0.6.g7505a.dirty
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html