Re: [RFC PATCH 5/5] builtin/branch: give more useful error messages when renaming

2017-09-23 Thread Kaartic Sivaraam
On Thursday 21 September 2017 02:27 AM, Stefan Beller wrote: It's conventional to report that 'tset' doesn't exist rather than reporting that 'master' exists, the same way the 'mv' command does. $ git branch -m tset master This is not the 'mv' command as promised? So this is just to

Re: [RFC PATCH 5/5] builtin/branch: give more useful error messages when renaming

2017-09-20 Thread Stefan Beller
On Tue, Sep 19, 2017 at 12:15 AM, Kaartic Sivaraam wrote: > When trying to rename an inexistent branch to an existing branch > the rename failed specifying the new branch name exists rather than > specifying that the branch trying to be renamed doesn't exist. > >

[RFC PATCH 5/5] builtin/branch: give more useful error messages when renaming

2017-09-19 Thread Kaartic Sivaraam
When trying to rename an inexistent branch to an existing branch the rename failed specifying the new branch name exists rather than specifying that the branch trying to be renamed doesn't exist. $ git branch -m tset master fatal: A branch named 'master' already exists. It's conventional