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-files.c | |
| parent | dee4e384f376020e08cb78f6dfaf00ae84e97a9e (diff) | |
| download | git-ef1d9c5aa4c8fd57b2a8043c0cd9fea1c507db6a.tar.gz git-ef1d9c5aa4c8fd57b2a8043c0cd9fea1c507db6a.tar.xz | |
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-files.c')
| -rw-r--r-- | builtin-diff-files.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin-diff-files.c b/builtin-diff-files.c index 81ac2fe64..2e1011862 100644 --- a/builtin-diff-files.c +++ b/builtin-diff-files.c @@ -18,8 +18,8 @@ int cmd_diff_files(int argc, const char **argv, char **envp) struct rev_info rev; int silent = 0; - git_config(git_default_config); /* no "diff" UI options */ init_revisions(&rev); + git_config(git_default_config); /* no "diff" UI options */ rev.abbrev = 0; argc = setup_revisions(argc, argv, &rev, NULL); |