wt-status: take advice.statusHints seriously
Currently, status gives a lot of hints even when advice.statusHints is false. Change this so that all hints depend on the config variable. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
18f3b5a9d3
commit
980bde3894
@ -151,7 +151,7 @@ cat >expect <<EOF
|
||||
# Changed but not updated:
|
||||
# modified: dir1/modified
|
||||
#
|
||||
# Untracked files not listed (use -u option to show untracked files)
|
||||
# Untracked files not listed
|
||||
EOF
|
||||
git config advice.statusHints false
|
||||
test_expect_success 'status -uno (advice.statusHints false)' '
|
||||
|
21
wt-status.c
21
wt-status.c
@ -625,7 +625,9 @@ void wt_status_print(struct wt_status *s)
|
||||
if (s->show_untracked_files)
|
||||
wt_status_print_untracked(s);
|
||||
else if (s->commitable)
|
||||
fprintf(s->fp, "# Untracked files not listed (use -u option to show untracked files)\n");
|
||||
fprintf(s->fp, "# Untracked files not listed%s\n",
|
||||
advice_status_hints
|
||||
? " (use -u option to show untracked files)" : "");
|
||||
|
||||
if (s->verbose)
|
||||
wt_status_print_verbose(s);
|
||||
@ -635,15 +637,22 @@ void wt_status_print(struct wt_status *s)
|
||||
else if (s->nowarn)
|
||||
; /* nothing */
|
||||
else if (s->workdir_dirty)
|
||||
printf("no changes added to commit (use \"git add\" and/or \"git commit -a\")\n");
|
||||
printf("no changes added to commit%s\n",
|
||||
advice_status_hints
|
||||
? " (use \"git add\" and/or \"git commit -a\")" : "");
|
||||
else if (s->untracked.nr)
|
||||
printf("nothing added to commit but untracked files present (use \"git add\" to track)\n");
|
||||
printf("nothing added to commit but untracked files present%s\n",
|
||||
advice_status_hints
|
||||
? " (use \"git add\" to track)" : "");
|
||||
else if (s->is_initial)
|
||||
printf("nothing to commit (create/copy files and use \"git add\" to track)\n");
|
||||
printf("nothing to commit%s\n", advice_status_hints
|
||||
? " (create/copy files and use \"git add\" to track)" : "");
|
||||
else if (!s->show_untracked_files)
|
||||
printf("nothing to commit (use -u to show untracked files)\n");
|
||||
printf("nothing to commit%s\n", advice_status_hints
|
||||
? " (use -u to show untracked files)" : "");
|
||||
else
|
||||
printf("nothing to commit (working directory clean)\n");
|
||||
printf("nothing to commit%s\n", advice_status_hints
|
||||
? " (working directory clean)" : "");
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user