diff options
author | Jean-Noel Avila <jn.avila@free.fr> | 2017-05-11 14:06:33 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-05-12 15:23:39 +0900 |
commit | 9932242f597b2990117430546e411793090e22e2 (patch) | |
tree | b237123d9502b84e5edbcd641060aa719b21696a /t/t5100/msg0005 | |
parent | usability: don't ask questions if no reply is required (diff) | |
download | tgif-9932242f597b2990117430546e411793090e22e2.tar.xz |
read-tree -m: make error message for merging 0 trees less smart aleck
"git read-tree -m" requires a tree argument to name the tree to be
merged in. Git uses a cutesy error message to say so and why:
$ git read-tree -m
warning: read-tree: emptying the index with no arguments is
deprecated; use --empty
fatal: just how do you expect me to merge 0 trees?
$ git read-tree -m --empty
fatal: just how do you expect me to merge 0 trees?
When lucky, that could produce an ah-hah moment for the user, but it's
more likely to irritate and distract them.
Instead, tell the user plainly that the tree argument is
required. Also document this requirement in the git-read-tree(1)
manpage where there is room to explain it in a more straightforward way.
Signed-off-by: Jean-Noel Avila <jn.avila@free.fr>
Helped-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t5100/msg0005')
0 files changed, 0 insertions, 0 deletions