On 8/25/08, Miklos Vajna <vmiklos@xxxxxxxxxxxxxx> wrote: > diff --git a/builtin-diff-index.c b/builtin-diff-index.c > index 17d851b..b8e0656 100644 > --- a/builtin-diff-index.c > +++ b/builtin-diff-index.c > @@ -16,6 +16,7 @@ int cmd_diff_index(int argc, const char **argv, const char *prefix) > int i; > int result; > > + setup_work_tree(); > init_revisions(&rev, prefix); > git_config(git_diff_basic_config, NULL); /* no "diff" UI options */ > rev.abbrev = 0; I think this is only needed when cached == 0 > diff --git a/builtin-diff.c b/builtin-diff.c > index 7ffea97..86f9255 100644 > --- a/builtin-diff.c > +++ b/builtin-diff.c > @@ -244,6 +244,7 @@ int cmd_diff(int argc, const char **argv, const char *prefix) > int nongit; > int result = 0; > > + setup_work_tree(); > /* > * We could get N tree-ish in the rev.pending_objects list. > * Also there could be M blobs there, and P pathspecs. > No. git-diff has too many modes, some does not need worktree. This forces worktree on all modes. -- Duy -- 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