Merge branch 'jc/show-untracked-false'

The status.showUntrackedFiles configuration variable had a name
that tempts users to set a Boolean value expressed in our usual
"false", "off", and "0", but it only took "no".  This has been
corrected so "true" and its synonyms are taken as "normal", while
"false" and its synonyms are taken as "no".

* jc/show-untracked-false:
  status: allow --untracked=false and friends
  status: unify parsing of --untracked= and status.showUntrackedFiles
This commit is contained in:
Junio C Hamano 2024-03-28 14:13:50 -07:00
commit bf0a352069
6 changed files with 68 additions and 30 deletions

View file

@ -57,6 +57,8 @@ status.showUntrackedFiles::
--
+
If this variable is not specified, it defaults to 'normal'.
All usual spellings for Boolean value `true` are taken as `normal`
and `false` as `no`.
This variable can be overridden with the -u|--untracked-files option
of linkgit:git-status[1] and linkgit:git-commit[1].

View file

@ -347,6 +347,8 @@ The possible options are:
- 'normal' - Shows untracked files and directories
- 'all' - Also shows individual files in untracked directories.
All usual spellings for Boolean value `true` are taken as `normal`
and `false` as `no`.
The default can be changed using the status.showUntrackedFiles
configuration variable documented in linkgit:git-config[1].
--

View file

@ -79,6 +79,8 @@ Consider enabling untracked cache and split index if supported (see
`git update-index --untracked-cache` and `git update-index
--split-index`), Otherwise you can use `no` to have `git status`
return more quickly without showing untracked files.
All usual spellings for Boolean value `true` are taken as `normal`
and `false` as `no`.
The default can be changed using the status.showUntrackedFiles
configuration variable documented in linkgit:git-config[1].

View file

@ -1157,22 +1157,45 @@ static void handle_ignored_arg(struct wt_status *s)
die(_("Invalid ignored mode '%s'"), ignored_arg);
}
static void handle_untracked_files_arg(struct wt_status *s)
static enum untracked_status_type parse_untracked_setting_name(const char *u)
{
if (!untracked_files_arg)
; /* default already initialized */
else if (!strcmp(untracked_files_arg, "no"))
s->show_untracked_files = SHOW_NO_UNTRACKED_FILES;
else if (!strcmp(untracked_files_arg, "normal"))
s->show_untracked_files = SHOW_NORMAL_UNTRACKED_FILES;
else if (!strcmp(untracked_files_arg, "all"))
s->show_untracked_files = SHOW_ALL_UNTRACKED_FILES;
/*
* Please update $__git_untracked_file_modes in
* git-completion.bash when you add new options
*/
switch (git_parse_maybe_bool(u)) {
case 0:
u = "no";
break;
case 1:
u = "normal";
break;
default:
break;
}
if (!strcmp(u, "no"))
return SHOW_NO_UNTRACKED_FILES;
else if (!strcmp(u, "normal"))
return SHOW_NORMAL_UNTRACKED_FILES;
else if (!strcmp(u, "all"))
return SHOW_ALL_UNTRACKED_FILES;
else
die(_("Invalid untracked files mode '%s'"), untracked_files_arg);
return SHOW_UNTRACKED_FILES_ERROR;
}
static void handle_untracked_files_arg(struct wt_status *s)
{
enum untracked_status_type u;
if (!untracked_files_arg)
return; /* default already initialized */
u = parse_untracked_setting_name(untracked_files_arg);
if (u == SHOW_UNTRACKED_FILES_ERROR)
die(_("Invalid untracked files mode '%s'"),
untracked_files_arg);
s->show_untracked_files = u;
}
static const char *read_commit_message(const char *name)
@ -1455,16 +1478,12 @@ static int git_status_config(const char *k, const char *v,
return 0;
}
if (!strcmp(k, "status.showuntrackedfiles")) {
if (!v)
return config_error_nonbool(k);
else if (!strcmp(v, "no"))
s->show_untracked_files = SHOW_NO_UNTRACKED_FILES;
else if (!strcmp(v, "normal"))
s->show_untracked_files = SHOW_NORMAL_UNTRACKED_FILES;
else if (!strcmp(v, "all"))
s->show_untracked_files = SHOW_ALL_UNTRACKED_FILES;
else
enum untracked_status_type u;
u = parse_untracked_setting_name(v);
if (u == SHOW_UNTRACKED_FILES_ERROR)
return error(_("Invalid untracked files mode '%s'"), v);
s->show_untracked_files = u;
return 0;
}
if (!strcmp(k, "diff.renamelimit")) {

View file

@ -419,14 +419,19 @@ Changes not staged for commit:
Untracked files not listed (use -u option to show untracked files)
EOF
git status -uno >output &&
test_cmp expect output &&
git status -ufalse >output &&
test_cmp expect output
'
test_expect_success 'status (status.showUntrackedFiles no)' '
test_config status.showuntrackedfiles no &&
git status >output &&
test_cmp expect output
'
for no in no false 0
do
test_expect_success "status (status.showUntrackedFiles $no)" '
test_config status.showuntrackedfiles "$no" &&
git status >output &&
test_cmp expect output
'
done
test_expect_success 'status -uno (advice.statusHints false)' '
cat >expect <<EOF &&
@ -488,14 +493,21 @@ Untracked files:
EOF
git status -unormal >output &&
test_cmp expect output &&
git status -utrue >output &&
test_cmp expect output &&
git status -uyes >output &&
test_cmp expect output
'
test_expect_success 'status (status.showUntrackedFiles normal)' '
test_config status.showuntrackedfiles normal &&
git status >output &&
test_cmp expect output
'
for normal in normal true 1
do
test_expect_success "status (status.showUntrackedFiles $normal)" '
test_config status.showuntrackedfiles $normal &&
git status >output &&
test_cmp expect output
'
done
cat >expect <<EOF
M dir1/modified

View file

@ -23,7 +23,8 @@ enum color_wt_status {
};
enum untracked_status_type {
SHOW_NO_UNTRACKED_FILES,
SHOW_UNTRACKED_FILES_ERROR = -1,
SHOW_NO_UNTRACKED_FILES = 0,
SHOW_NORMAL_UNTRACKED_FILES,
SHOW_ALL_UNTRACKED_FILES
};