diff options
| author | Johannes Schindelin <Johannes.Schindelin@gmx.de> | 2007-11-09 11:34:07 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2007-11-09 21:45:01 -0800 |
| commit | 354e6534c1569d0d23937add8e7b55bf87bbaf20 (patch) | |
| tree | e209ad157a72348fc7a715f08e0030089c347bb8 | |
| parent | 2729cadca22dce2b5bb994581c45a6a71daf7998 (diff) | |
| download | git-354e6534c1569d0d23937add8e7b55bf87bbaf20.tar.gz git-354e6534c1569d0d23937add8e7b55bf87bbaf20.tar.xz | |
builtin-blame: set up the work_tree before the first file access
We check in cmd_blame() if the specified path is there, but we
failed to set up the working tree before that.
While at it, make setup_work_tree() just return if it was run
before.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin-blame.c | 1 | ||||
| -rw-r--r-- | setup.c | 10 |
2 files changed, 9 insertions, 2 deletions
diff --git a/builtin-blame.c b/builtin-blame.c index 55a3c0bc5..ba80bf894 100644 --- a/builtin-blame.c +++ b/builtin-blame.c @@ -2295,6 +2295,7 @@ int cmd_blame(int argc, const char **argv, const char *prefix) else if (i != argc - 1) usage(blame_usage); /* garbage at end */ + setup_work_tree(); if (!has_path_in_work_tree(path)) die("cannot stat path %s: %s", path, strerror(errno)); @@ -208,12 +208,18 @@ static const char *set_work_tree(const char *dir) void setup_work_tree(void) { - const char *work_tree = get_git_work_tree(); - const char *git_dir = get_git_dir(); + const char *work_tree, *git_dir; + static int initialized = 0; + + if (initialized) + return; + work_tree = get_git_work_tree(); + git_dir = get_git_dir(); if (!is_absolute_path(git_dir)) set_git_dir(make_absolute_path(git_dir)); if (!work_tree || chdir(work_tree)) die("This operation must be run in a work tree"); + initialized = 1; } /* |