diff options
| author | Junio C Hamano <junkio@cox.net> | 2006-07-27 22:55:44 -0700 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2006-07-27 22:55:44 -0700 |
| commit | ef1d9c5aa4c8fd57b2a8043c0cd9fea1c507db6a (patch) | |
| tree | 8e5537ca7f1be31caad67d68310d0ec3e805b3af /builtin-diff-tree.c | |
| parent | dee4e384f376020e08cb78f6dfaf00ae84e97a9e (diff) | |
| download | git-ef1d9c5aa4c8fd57b2a8043c0cd9fea1c507db6a.tar.gz | |
log and diff family: honor config even from subdirectories
There currently is an unfortunate circular dependency between
what init_revisions (the command line revision specification
parser) does and setting up the log and diff options. The
function uses setup_git_directory() to find the root of the
project relative to the current directory and calls diff_setup()
to prepare diff generation. However, some of the things that
diff_setup() does needs to depend on the configuration variable,
which needs to be read after setup_git_directory() is called.
This patch is a low impact workaround. It first lets
init_revisions() to run and do its thing, then uses git_config()
and diff_setup() after it returns, so that configuration
variables that affects the diff operation can be used from
subdirectories.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-diff-tree.c')
| -rw-r--r-- | builtin-diff-tree.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin-diff-tree.c b/builtin-diff-tree.c index b610668594..8957b459de 100644 --- a/builtin-diff-tree.c +++ b/builtin-diff-tree.c @@ -67,9 +67,9 @@ int cmd_diff_tree(int argc, const char **argv, char **envp) static struct rev_info *opt = &log_tree_opt; int read_stdin = 0; + init_revisions(opt); git_config(git_default_config); /* no "diff" UI options */ nr_sha1 = 0; - init_revisions(opt); opt->abbrev = 0; opt->diff = 1; argc = setup_revisions(argc, argv, opt, NULL); |