diff options
| author | Junio C Hamano <gitster@pobox.com> | 2008-07-20 00:21:38 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2008-07-20 17:21:32 -0700 |
| commit | d14e7407b34b8c11b407358b9677c283c381a71c (patch) | |
| tree | 2ea98209de908a49d8841a7fc6ccb1e5b186795e /t/t7102-reset.sh | |
| parent | 559e840b74dbbf39633aa68627937a5f7a6d4145 (diff) | |
| download | git-d14e7407b34b8c11b407358b9677c283c381a71c.tar.gz | |
"needs update" considered harmful
"git update-index --refresh", "git reset" and "git add --refresh" have
reported paths that have local modifications as "needs update" since the
beginning of git.
Although this is logically correct in that you need to update the index at
that path before you can commit that change, it is now becoming more and
more clear, especially with the continuous push for user friendliness
since 1.5.0 series, that the message is suboptimal. After all, the change
may be something the user might want to get rid of, and "updating" would
be absolutely a wrong thing to do if that is the case.
I prepared two alternatives to solve this. Both aim to reword the message
to more neutral "locally modified".
This patch is a more intrusive variant that changes the message for only
Porcelain commands ("add" and "reset") while keeping the plumbing
"update-index" intact.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t7102-reset.sh')
| -rwxr-xr-x | t/t7102-reset.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/t/t7102-reset.sh b/t/t7102-reset.sh index 0bfc1e9be4..29f5678b4c 100755 --- a/t/t7102-reset.sh +++ b/t/t7102-reset.sh @@ -419,7 +419,7 @@ test_expect_success 'resetting an unmodified path is a no-op' ' ' cat > expect << EOF -file2: needs update +file2: locally modified EOF test_expect_success '--mixed refreshes the index' ' |