Skip to content

Commit f4aed1f

Browse files
committed
Merge branch 'core-longpaths-everywhere'
Git for Windows supports the core.longPaths config setting to allow writing/reading long paths via the \\?\ trick for a long time now. However, for that support to work, it is absolutely necessary that git_default_config() is given a chance to parse the config. Otherwise Git will be non the wiser. So let's make sure that as many commands that previously failed to parse the core.* settings now do that, implicitly enabling long path support in a lot more places. Note: this is not a perfect solution, and it cannot be, as there is a chicken-and-egg problem in reading the config itself... This fixes #1218 Signed-off-by: Johannes Schindelin <[email protected]>
2 parents 8152b88 + 3085c9e commit f4aed1f

34 files changed

+64
-5
lines changed

builtin/archive.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
#include "parse-options.h"
1010
#include "pkt-line.h"
1111
#include "sideband.h"
12+
#include "config.h"
1213

1314
static void create_output_file(const char *output_file)
1415
{
@@ -94,6 +95,7 @@ int cmd_archive(int argc, const char **argv, const char *prefix)
9495
OPT_END()
9596
};
9697

98+
git_config(git_default_config, NULL);
9799
argc = parse_options(argc, argv, prefix, local_opts, NULL,
98100
PARSE_OPT_KEEP_ALL);
99101

builtin/bisect--helper.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
#include "parse-options.h"
44
#include "bisect.h"
55
#include "refs.h"
6+
#include "config.h"
67

78
static GIT_PATH_FUNC(git_path_bisect_terms, "BISECT_TERMS")
89
static GIT_PATH_FUNC(git_path_bisect_expected_rev, "BISECT_EXPECTED_REV")
@@ -129,6 +130,7 @@ int cmd_bisect__helper(int argc, const char **argv, const char *prefix)
129130
OPT_END()
130131
};
131132

133+
git_config(git_default_config, NULL);
132134
argc = parse_options(argc, argv, prefix, options,
133135
git_bisect_helper_usage, 0);
134136

builtin/bundle.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
#include "builtin.h"
22
#include "cache.h"
33
#include "bundle.h"
4+
#include "config.h"
45

56
/*
67
* Basic handler for bundle files to connect repositories via sneakernet.
@@ -21,6 +22,7 @@ int cmd_bundle(int argc, const char **argv, const char *prefix)
2122
const char *cmd, *bundle_file;
2223
int bundle_fd = -1;
2324

25+
git_config(git_default_config, NULL);
2426
if (argc < 3)
2527
usage(builtin_bundle_usage);
2628

builtin/check-ref-format.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
#include "refs.h"
77
#include "builtin.h"
88
#include "strbuf.h"
9+
#include "config.h"
910

1011
static const char builtin_check_ref_format_usage[] =
1112
"git check-ref-format [--normalize] [<options>] <refname>\n"
@@ -58,6 +59,7 @@ int cmd_check_ref_format(int argc, const char **argv, const char *prefix)
5859
int flags = 0;
5960
const char *refname;
6061

62+
git_config(git_default_config, NULL);
6163
if (argc == 2 && !strcmp(argv[1], "-h"))
6264
usage(builtin_check_ref_format_usage);
6365

builtin/clone.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -897,6 +897,7 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
897897

898898
struct refspec_item refspec;
899899

900+
git_config(platform_core_config, NULL);
900901
fetch_if_missing = 0;
901902

902903
packet_trace_identity("clone");

builtin/column.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@ int cmd_column(int argc, const char **argv, const char *prefix)
3434
OPT_END()
3535
};
3636

37+
git_config(platform_core_config, NULL);
38+
3739
/* This one is special and must be the first one */
3840
if (argc > 1 && starts_with(argv[1], "--command=")) {
3941
command = argv[1] + 10;

builtin/credential.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
#include "git-compat-util.h"
22
#include "credential.h"
33
#include "builtin.h"
4+
#include "config.h"
45

56
static const char usage_msg[] =
67
"git credential [fill|approve|reject]";
@@ -10,6 +11,8 @@ int cmd_credential(int argc, const char **argv, const char *prefix)
1011
const char *op;
1112
struct credential c = CREDENTIAL_INIT;
1213

14+
git_config(git_default_config, NULL);
15+
1316
if (argc != 2 || !strcmp(argv[1], "-h"))
1417
usage(usage_msg);
1518
op = argv[1];

builtin/fetch-pack.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
#include "connect.h"
66
#include "sha1-array.h"
77
#include "protocol.h"
8+
#include "config.h"
89

910
static const char fetch_pack_usage[] =
1011
"git fetch-pack [--all] [--stdin] [--quiet | -q] [--keep | -k] [--thin] "
@@ -55,6 +56,7 @@ int cmd_fetch_pack(int argc, const char **argv, const char *prefix)
5556
struct string_list deepen_not = STRING_LIST_INIT_DUP;
5657
struct packet_reader reader;
5758

59+
git_config(git_default_config, NULL);
5860
fetch_if_missing = 0;
5961

6062
packet_trace_identity("fetch-pack");

builtin/get-tar-commit-id.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
#include "tar.h"
77
#include "builtin.h"
88
#include "quote.h"
9+
#include "config.h"
910

1011
static const char builtin_get_tar_commit_id_usage[] =
1112
"git get-tar-commit-id";
@@ -25,6 +26,7 @@ int cmd_get_tar_commit_id(int argc, const char **argv, const char *prefix)
2526
if (argc != 1)
2627
usage(builtin_get_tar_commit_id_usage);
2728

29+
git_config(git_default_config, NULL);
2830
n = read_in_full(0, buffer, HEADERSIZE);
2931
if (n < 0)
3032
die_errno("git get-tar-commit-id: read error");

builtin/interpret-trailers.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
#include "parse-options.h"
1111
#include "string-list.h"
1212
#include "trailer.h"
13+
#include "config.h"
1314

1415
static const char * const git_interpret_trailers_usage[] = {
1516
N_("git interpret-trailers [--in-place] [--trim-empty] [(--trailer <token>[(=|:)<value>])...] [<file>...]"),
@@ -109,6 +110,7 @@ int cmd_interpret_trailers(int argc, const char **argv, const char *prefix)
109110
OPT_END()
110111
};
111112

113+
git_config(git_default_config, NULL);
112114
argc = parse_options(argc, argv, prefix, options,
113115
git_interpret_trailers_usage, 0);
114116

builtin/log.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1902,6 +1902,7 @@ int cmd_cherry(int argc, const char **argv, const char *prefix)
19021902
OPT_END()
19031903
};
19041904

1905+
git_config(git_default_config, NULL);
19051906
argc = parse_options(argc, argv, prefix, options, cherry_usage, 0);
19061907

19071908
switch (argc) {

builtin/ls-remote.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
#include "ref-filter.h"
55
#include "remote.h"
66
#include "refs.h"
7+
#include "config.h"
78

89
static const char * const ls_remote_usage[] = {
910
N_("git ls-remote [--heads] [--tags] [--refs] [--upload-pack=<exec>]\n"
@@ -84,6 +85,7 @@ int cmd_ls_remote(int argc, const char **argv, const char *prefix)
8485
PARSE_OPT_STOP_AT_NON_OPTION);
8586
dest = argv[0];
8687

88+
git_config(git_default_config, NULL);
8789
if (argc > 1) {
8890
int i;
8991
pattern = xcalloc(argc, sizeof(const char *));

builtin/mailinfo.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
#include "utf8.h"
88
#include "strbuf.h"
99
#include "mailinfo.h"
10+
#include "config.h"
1011

1112
static const char mailinfo_usage[] =
1213
"git mailinfo [-k | -b] [-m | --message-id] [-u | --encoding=<encoding> | -n] [--scissors | --no-scissors] <msg> <patch> < mail >info";
@@ -18,6 +19,7 @@ int cmd_mailinfo(int argc, const char **argv, const char *prefix)
1819
int status;
1920
char *msgfile, *patchfile;
2021

22+
git_config(git_default_config, NULL);
2123
setup_mailinfo(&mi);
2224

2325
def_charset = get_commit_output_encoding();

builtin/mailsplit.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
#include "builtin.h"
99
#include "string-list.h"
1010
#include "strbuf.h"
11+
#include "config.h"
1112

1213
static const char git_mailsplit_usage[] =
1314
"git mailsplit [-d<prec>] [-f<n>] [-b] [--keep-cr] -o<directory> [(<mbox>|<Maildir>)...]";
@@ -276,6 +277,7 @@ int cmd_mailsplit(int argc, const char **argv, const char *prefix)
276277
const char **argp;
277278
static const char *stdin_only[] = { "-", NULL };
278279

280+
git_config(git_default_config, NULL);
279281
for (argp = argv+1; *argp; argp++) {
280282
const char *arg = *argp;
281283

builtin/merge-index.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
#include "builtin.h"
22
#include "run-command.h"
3+
#include "config.h"
34

45
static const char *pgm;
56
static int one_shot, quiet;
@@ -74,6 +75,8 @@ int cmd_merge_index(int argc, const char **argv, const char *prefix)
7475
*/
7576
signal(SIGCHLD, SIG_DFL);
7677

78+
git_config(git_default_config, NULL);
79+
7780
if (argc < 3)
7881
usage("git merge-index [-o] [-q] <merge-program> (-a | [--] [<filename>...])");
7982

builtin/merge-tree.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
#include "blob.h"
55
#include "exec-cmd.h"
66
#include "merge-blobs.h"
7+
#include "config.h"
78

89
static const char merge_tree_usage[] = "git merge-tree <base-tree> <branch1> <branch2>";
910

@@ -367,6 +368,7 @@ int cmd_merge_tree(int argc, const char **argv, const char *prefix)
367368
if (argc != 4)
368369
usage(merge_tree_usage);
369370

371+
git_config(git_default_config, NULL);
370372
buf1 = get_tree_descriptor(t+0, argv[1]);
371373
buf2 = get_tree_descriptor(t+1, argv[2]);
372374
buf3 = get_tree_descriptor(t+2, argv[3]);

builtin/mktag.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
#include "builtin.h"
22
#include "tag.h"
33
#include "replace-object.h"
4+
#include "config.h"
45

56
/*
67
* A signature file has a very simple fixed format: four lines
@@ -157,6 +158,7 @@ int cmd_mktag(int argc, const char **argv, const char *prefix)
157158
if (argc != 1)
158159
usage("git mktag");
159160

161+
git_config(git_default_config, NULL);
160162
if (strbuf_read(&buf, 0, 4096) < 0) {
161163
die_errno("could not read from stdin");
162164
}

builtin/mktree.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
#include "quote.h"
88
#include "tree.h"
99
#include "parse-options.h"
10+
#include "config.h"
1011

1112
static struct treeent {
1213
unsigned mode;
@@ -156,6 +157,7 @@ int cmd_mktree(int ac, const char **av, const char *prefix)
156157
OPT_END()
157158
};
158159

160+
git_config(git_default_config, NULL);
159161
ac = parse_options(ac, av, prefix, option, mktree_usage, 0);
160162
getline_fn = nul_term_line ? strbuf_getline_nul : strbuf_getline_lf;
161163

builtin/pack-refs.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
#include "parse-options.h"
33
#include "refs.h"
44
#include "repository.h"
5+
#include "config.h"
56

67
static char const * const pack_refs_usage[] = {
78
N_("git pack-refs [<options>]"),
@@ -16,6 +17,7 @@ int cmd_pack_refs(int argc, const char **argv, const char *prefix)
1617
OPT_BIT(0, "prune", &flags, N_("prune loose refs (default)"), PACK_REFS_PRUNE),
1718
OPT_END(),
1819
};
20+
git_config(git_default_config, NULL);
1921
if (parse_options(argc, argv, prefix, opts, pack_refs_usage, 0))
2022
usage_with_options(pack_refs_usage, opts);
2123
return refs_pack_refs(get_main_ref_store(the_repository), flags);

builtin/prune-packed.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
#include "progress.h"
44
#include "parse-options.h"
55
#include "packfile.h"
6+
#include "config.h"
67

78
static const char * const prune_packed_usage[] = {
89
N_("git prune-packed [-n | --dry-run] [-q | --quiet]"),
@@ -59,6 +60,7 @@ int cmd_prune_packed(int argc, const char **argv, const char *prefix)
5960
OPT_END()
6061
};
6162

63+
git_config(git_default_config, NULL);
6264
argc = parse_options(argc, argv, prefix, prune_packed_options,
6365
prune_packed_usage, 0);
6466

builtin/prune.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
#include "reachable.h"
77
#include "parse-options.h"
88
#include "progress.h"
9+
#include "config.h"
910

1011
static const char * const prune_usage[] = {
1112
N_("git prune [-n] [-v] [--progress] [--expire <time>] [--] [<head>...]"),
@@ -115,6 +116,8 @@ int cmd_prune(int argc, const char **argv, const char *prefix)
115116
};
116117
char *s;
117118

119+
git_config(git_default_config, NULL);
120+
118121
expire = TIME_MAX;
119122
save_commit_buffer = 0;
120123
check_replace_refs = 0;

builtin/reflog.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -719,6 +719,7 @@ static const char reflog_usage[] =
719719

720720
int cmd_reflog(int argc, const char **argv, const char *prefix)
721721
{
722+
git_config(git_default_config, NULL);
722723
if (argc > 1 && !strcmp(argv[1], "-h"))
723724
usage(reflog_usage);
724725

builtin/remote-ext.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
#include "transport.h"
33
#include "run-command.h"
44
#include "pkt-line.h"
5+
#include "config.h"
56

67
static const char usage_msg[] =
78
"git remote-ext <remote> <url>";
@@ -198,5 +199,6 @@ int cmd_remote_ext(int argc, const char **argv, const char *prefix)
198199
if (argc != 3)
199200
usage(usage_msg);
200201

202+
git_config(git_default_config, NULL);
201203
return command_loop(argv[2]);
202204
}

builtin/remote.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1609,6 +1609,7 @@ int cmd_remote(int argc, const char **argv, const char *prefix)
16091609
};
16101610
int result;
16111611

1612+
git_config(git_default_config, NULL);
16121613
argc = parse_options(argc, argv, prefix, options, builtin_remote_usage,
16131614
PARSE_OPT_STOP_AT_NON_OPTION);
16141615

builtin/rev-parse.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -423,6 +423,7 @@ static int cmd_parseopt(int argc, const char **argv, const char *prefix)
423423
struct option *opts = NULL;
424424
int onb = 0, osz = 0, unb = 0, usz = 0;
425425

426+
git_config(git_default_config, NULL);
426427
strbuf_addstr(&parsed, "set --");
427428
argc = parse_options(argc, argv, prefix, parseopt_opts, parseopt_usage,
428429
PARSE_OPT_KEEP_DASHDASH);

builtin/send-pack.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ static int send_pack_config(const char *k, const char *v, void *cb)
121121
}
122122
}
123123
}
124-
return 0;
124+
return git_default_config(k, v, cb);
125125
}
126126

127127
int cmd_send_pack(int argc, const char **argv, const char *prefix)

builtin/show-ref.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
#include "tag.h"
66
#include "string-list.h"
77
#include "parse-options.h"
8+
#include "config.h"
89

910
static const char * const show_ref_usage[] = {
1011
N_("git show-ref [-q | --quiet] [--verify] [--head] [-d | --dereference] [-s | --hash[=<n>]] [--abbrev[=<n>]] [--tags] [--heads] [--] [<pattern>...]"),
@@ -180,6 +181,7 @@ static const struct option show_ref_options[] = {
180181

181182
int cmd_show_ref(int argc, const char **argv, const char *prefix)
182183
{
184+
git_config(git_default_config, NULL);
183185
argc = parse_options(argc, argv, prefix, show_ref_options,
184186
show_ref_usage, 0);
185187

builtin/stripspace.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,9 @@ int cmd_stripspace(int argc, const char **argv, const char *prefix)
4545
if (argc)
4646
usage_with_options(stripspace_usage, options);
4747

48-
if (mode == STRIP_COMMENTS || mode == COMMENT_LINES) {
48+
if (mode == STRIP_COMMENTS || mode == COMMENT_LINES)
4949
setup_git_directory_gently(NULL);
50-
git_config(git_default_config, NULL);
51-
}
50+
git_config(git_default_config, NULL);
5251

5352
if (strbuf_read(&buf, 0, 1024) < 0)
5453
die_errno("could not read the input");

builtin/submodule--helper.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1891,6 +1891,7 @@ static struct cmd_struct commands[] = {
18911891
int cmd_submodule__helper(int argc, const char **argv, const char *prefix)
18921892
{
18931893
int i;
1894+
git_config(git_default_config, NULL);
18941895
if (argc < 2 || !strcmp(argv[1], "-h"))
18951896
usage("git submodule--helper <command>");
18961897

0 commit comments

Comments
 (0)