Skip to content

Commit 5f38630

Browse files
committed
Windows: add support for a Windows-wide configuration
Between the libgit2 and the Git for Windows project, there has been a discussion how we could share Git configuration to avoid duplication (or worse: skew). Earlier, libgit2 was nice enough to just re-use Git for Windows' C:\Program Files (x86)\Git\etc\gitconfig but with the upcoming Git for Windows 2.x, there would be more paths to search, as we will have 64-bit and 32-bit versions, and the corresponding config files will be in %PROGRAMFILES%\Git\mingw64\etc and ...\mingw32\etc, respectively. Worse: there are portable Git for Windows versions out there which live in totally unrelated directories, still. Therefore we came to a consensus to use `%PROGRAMDATA%\Git\config` as the location for shared Git settings that are of wider interest than just Git for Windows. On XP, there is no %PROGRAMDATA%, therefore we need to use "%ALLUSERSPROFILE%\Application Data\Git\config" in those setups. Of course, the configuration in `%PROGRAMDATA%\Git\config` has the widest reach, therefore it must take the lowest precedence, i.e. Git for Windows can still override settings in its `etc/gitconfig` file. Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 672159f commit 5f38630

File tree

7 files changed

+39
-3
lines changed

7 files changed

+39
-3
lines changed

Documentation/config.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,9 @@ the Git commands' behavior. The `.git/config` file in each repository
66
is used to store the configuration for that repository, and
77
`$HOME/.gitconfig` is used to store a per-user configuration as
88
fallback values for the `.git/config` file. The file `/etc/gitconfig`
9-
can be used to store a system-wide default configuration.
9+
can be used to store a system-wide default configuration. On Windows,
10+
configuration can also be stored in `C:\ProgramData\Git\config`; This
11+
file will be used also by libgit2-based software.
1012

1113
The configuration variables are used by both the Git plumbing
1214
and the porcelains. The variables are divided into sections, wherein

Documentation/git-config.txt

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -246,6 +246,11 @@ $XDG_CONFIG_HOME/git/config::
246246
$GIT_DIR/config::
247247
Repository specific configuration file.
248248

249+
On Windows, as there is no central `/etc/` directory, there is yet another
250+
config file, intended to contain settings for *all* Git-related software
251+
running on the machine. Consequently, this config file takes an even lower
252+
precedence than the `$(prefix)/etc/gitconfig` file.
253+
249254
If no further options are given, all reading options will read all of these
250255
files that are available. If the global or the system-wide configuration
251256
file are not available they will be ignored. If the repository configuration

Documentation/git.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -986,7 +986,8 @@ for further details.
986986

987987
'GIT_CONFIG_NOSYSTEM'::
988988
Whether to skip reading settings from the system-wide
989-
`$(prefix)/etc/gitconfig` file. This environment variable can
989+
`$(prefix)/etc/gitconfig` file (and on Windows, also from the
990+
`%PROGRAMDATA%\Git\config` file). This environment variable can
990991
be used along with `$HOME` and `$XDG_CONFIG_HOME` to create a
991992
predictable environment for a picky script, or you can set it
992993
temporarily to avoid using a buggy `/etc/gitconfig` file while

compat/mingw.c

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2464,3 +2464,22 @@ int uname(struct utsname *buf)
24642464
"%u", (v >> 16) & 0x7fff);
24652465
return 0;
24662466
}
2467+
2468+
const char *program_data_config(void)
2469+
{
2470+
static struct strbuf path = STRBUF_INIT;
2471+
static unsigned initialized;
2472+
2473+
if (!initialized) {
2474+
const char *env = mingw_getenv("PROGRAMDATA");
2475+
const char *extra = "";
2476+
if (!env) {
2477+
env = mingw_getenv("ALLUSERSPROFILE");
2478+
extra = "/Application Data";
2479+
}
2480+
if (env)
2481+
strbuf_addf(&path, "%s%s/Git/config", env, extra);
2482+
initialized = 1;
2483+
}
2484+
return *path.buf ? path.buf : NULL;
2485+
}

compat/mingw.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -420,6 +420,8 @@ static inline char *mingw_find_last_dir_sep(const char *path)
420420
int mingw_offset_1st_component(const char *path);
421421
#define offset_1st_component mingw_offset_1st_component
422422
#define PATH_SEP ';'
423+
extern const char *program_data_config(void);
424+
#define git_program_data_config program_data_config
423425
#ifndef __MINGW64_VERSION_MAJOR
424426
#define PRIuMAX "I64u"
425427
#define PRId64 "I64d"

config.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1218,9 +1218,12 @@ int git_config_early(config_fn_t fn, void *data, const char *repo_config)
12181218
char *xdg_config = xdg_config_home("config");
12191219
char *user_config = expand_user_path("~/.gitconfig");
12201220

1221-
if (git_config_system())
1221+
if (git_config_system()) {
1222+
config_from_file_gently(fn, git_program_data_config(), data,
1223+
0, &ret, &found);
12221224
config_from_file_gently(fn, git_etc_gitconfig(), data, 0,
12231225
&ret, &found);
1226+
}
12241227

12251228
config_from_file_gently(fn, xdg_config, data, ACCESS_EACCES_OK,
12261229
&ret, &found);

git-compat-util.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -371,6 +371,10 @@ static inline char *git_find_last_dir_sep(const char *path)
371371
#define find_last_dir_sep git_find_last_dir_sep
372372
#endif
373373

374+
#ifndef git_program_data_config
375+
#define git_program_data_config() NULL
376+
#endif
377+
374378
#if defined(__HP_cc) && (__HP_cc >= 61000)
375379
#define NORETURN __attribute__((noreturn))
376380
#define NORETURN_PTR

0 commit comments

Comments
 (0)