Usually, git calls some form of setup_git_directory at startup. But sometimes, it doesn't. Usually, that's OK because it's not really using the repository. But in some cases, it is using the repo. In those cases, either setup_git_directory_gently must be called, or the repository (e.g. the refs) must not be accessed.
In every case except grep and shortlog, we fix this problem by making the call. In grep, in the --no-index mode, we don't want to access repository, so we set a flag which prevents this. In shortlog, we only want to skip accessing the repository when running without a repo (in stdin mode), so we check that before calling the only repo-dependent function that doesn't do its own setup. Suggested-by: Jeff King <p...@peff.net> Signed-off-by: David Turner <dtur...@twopensource.com> --- builtin/grep.c | 1 + builtin/log.c | 2 +- builtin/shortlog.c | 7 ++++--- git.c | 2 +- shortlog.h | 2 +- test-match-trees.c | 2 ++ test-revision-walking.c | 2 ++ 7 files changed, 12 insertions(+), 6 deletions(-) diff --git a/builtin/grep.c b/builtin/grep.c index 9e3f1cf..1e36b52 100644 --- a/builtin/grep.c +++ b/builtin/grep.c @@ -531,6 +531,7 @@ static int grep_directory(struct grep_opt *opt, const struct pathspec *pathspec, if (exc_std) setup_standard_excludes(&dir); + dir.flags |= DIR_NO_GITLINKS; fill_directory(&dir, pathspec); for (i = 0; i < dir.nr; i++) { if (!dir_path_match(dir.entries[i], pathspec, 0, NULL)) diff --git a/builtin/log.c b/builtin/log.c index 0d738d6..1d0e43e 100644 --- a/builtin/log.c +++ b/builtin/log.c @@ -975,7 +975,7 @@ static void make_cover_letter(struct rev_info *rev, int use_stdout, strbuf_release(&sb); - shortlog_init(&log); + shortlog_init(&log, 0); log.wrap_lines = 1; log.wrap = 72; log.in1 = 2; diff --git a/builtin/shortlog.c b/builtin/shortlog.c index bfc082e..ab4305b 100644 --- a/builtin/shortlog.c +++ b/builtin/shortlog.c @@ -219,11 +219,12 @@ static int parse_wrap_args(const struct option *opt, const char *arg, int unset) return 0; } -void shortlog_init(struct shortlog *log) +void shortlog_init(struct shortlog *log, int nongit) { memset(log, 0, sizeof(*log)); - read_mailmap(&log->mailmap, &log->common_repo_prefix); + if (!nongit) + read_mailmap(&log->mailmap, &log->common_repo_prefix); log->list.strdup_strings = 1; log->wrap = DEFAULT_WRAPLEN; @@ -252,7 +253,7 @@ int cmd_shortlog(int argc, const char **argv, const char *prefix) struct parse_opt_ctx_t ctx; git_config(git_default_config, NULL); - shortlog_init(&log); + shortlog_init(&log, nongit); init_revisions(&rev, prefix); parse_options_start(&ctx, argc, argv, prefix, options, PARSE_OPT_KEEP_DASHDASH | PARSE_OPT_KEEP_ARGV0); diff --git a/git.c b/git.c index 6cc0c07..51e0508 100644 --- a/git.c +++ b/git.c @@ -376,7 +376,7 @@ static struct cmd_struct commands[] = { { "am", cmd_am, RUN_SETUP | NEED_WORK_TREE }, { "annotate", cmd_annotate, RUN_SETUP }, { "apply", cmd_apply, RUN_SETUP_GENTLY }, - { "archive", cmd_archive }, + { "archive", cmd_archive, RUN_SETUP_GENTLY }, { "bisect--helper", cmd_bisect__helper, RUN_SETUP }, { "blame", cmd_blame, RUN_SETUP }, { "branch", cmd_branch, RUN_SETUP }, diff --git a/shortlog.h b/shortlog.h index de4f86f..ed1fbca 100644 --- a/shortlog.h +++ b/shortlog.h @@ -19,7 +19,7 @@ struct shortlog { struct string_list mailmap; }; -void shortlog_init(struct shortlog *log); +void shortlog_init(struct shortlog *log, int nongit); void shortlog_add_commit(struct shortlog *log, struct commit *commit); diff --git a/test-match-trees.c b/test-match-trees.c index 109f03e..4dad709 100644 --- a/test-match-trees.c +++ b/test-match-trees.c @@ -6,6 +6,8 @@ int main(int ac, char **av) unsigned char hash1[20], hash2[20], shifted[20]; struct tree *one, *two; + setup_git_directory(); + if (get_sha1(av[1], hash1)) die("cannot parse %s as an object name", av[1]); if (get_sha1(av[2], hash2)) diff --git a/test-revision-walking.c b/test-revision-walking.c index 285f06b..3d03133 100644 --- a/test-revision-walking.c +++ b/test-revision-walking.c @@ -50,6 +50,8 @@ int main(int argc, char **argv) if (argc < 2) return 1; + setup_git_directory(); + if (!strcmp(argv[1], "run-twice")) { printf("1st\n"); if (!run_revision_walk()) -- 2.4.2.767.g62658d5-twtrsrc -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html