Skip to content

Commit 534376c

Browse files
peffgitster
authored andcommitted
mv: be quiet about overwriting
When a user asks us to force a mv and overwrite the destination, we print a warning. However, since a typical use would be: $ git mv one two fatal: destination exists, source=one, destination=two $ git mv -f one two warning: overwriting 'two' this warning is just noise. We already know we're overwriting; that's why we gave -f! This patch silences the warning unless "--verbose" is given. Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent cd40b05 commit 534376c

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

builtin/mv.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,8 @@ int cmd_mv(int argc, const char **argv, const char *prefix)
173173
* check both source and destination
174174
*/
175175
if (S_ISREG(st.st_mode) || S_ISLNK(st.st_mode)) {
176-
warning(_("overwriting '%s'"), dst);
176+
if (verbose)
177+
warning(_("overwriting '%s'"), dst);
177178
bad = NULL;
178179
} else
179180
bad = _("Cannot overwrite");

0 commit comments

Comments
 (0)