Skip to content

Commit

Permalink
Merge branch 'jc/core-checkstat'
Browse files Browse the repository at this point in the history
The configuration variable core.checkstat was advertised in the
documentation but the code expected core.statinfo instead.

For now, we accept both core.checkstat and core.statinfo, but the
latter will be removed in the longer term.

* jc/core-checkstat:
  deprecate core.statinfo at Git 2.0 boundary
  • Loading branch information
Junio C Hamano committed Jun 5, 2013
2 parents 2fe2458 + 6f4dd60 commit 446913e
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion config.c
Original file line number Diff line number Diff line change
Expand Up @@ -566,7 +566,20 @@ static int git_default_core_config(const char *var, const char *value)
trust_ctime = git_config_bool(var, value);
return 0;
}
if (!strcmp(var, "core.statinfo")) {
if (!strcmp(var, "core.statinfo") ||
!strcmp(var, "core.checkstat")) {
/*
* NEEDSWORK: statinfo was a typo in v1.8.2 that has
* never been advertised. we will remove it at Git
* 2.0 boundary.
*/
if (!strcmp(var, "core.statinfo")) {
static int warned;
if (!warned++) {
warning("'core.statinfo' will be removed in Git 2.0; "
"use 'core.checkstat' instead.");
}
}
if (!strcasecmp(value, "default"))
check_stat = 1;
else if (!strcasecmp(value, "minimal"))
Expand Down

0 comments on commit 446913e

Please sign in to comment.