[PATCH V3] push: provide a hint when no paths in configured (issue3692)

Mads Kiilerich mads at kiilerich.com
Sun Feb 23 14:37:23 CST 2014


On 02/23/2014 12:02 AM, Anurag Goel wrote:
> Apologize for the mistake of "more than 80 characters" in one line.

And for top posting? :-) http://mercurial.selenic.com/wiki/MailingLists

> About the duplication of the 'other =' line, i don't think always 
> catching RepoError would be a good idea.
> There are different RepoError with different abort messages.So if we 
> always catch RepoError then we have to handle each error message 
> explicitly.
> For eg. abort message by "RepoError"  in file "mercurial/sshpeer.py" 
> and "mercurial/localrepo.py" is different.

Yes, thus the suggestion to re-raise the exception if it not is from a 
default-push dest.

/Mads



More information about the Mercurial-devel mailing list