<div dir="ltr"><div>I made the changes in the program as you suggested.<br><br>For the following patch , i am still getting into loop when type "make test" but when i test each file individually, it works fine.<br>
<br></div><div>I guess now there is no coding style error then why is it so happening?<br></div><div><div><br><br>diff -r d98ba4a87427 mercurial/commands.py<br>--- a/mercurial/commands.py    Mon Feb 10 17:31:26 2014 -0600<br>
+++ b/mercurial/commands.py    Wed Feb 12 02:17:29 2014 +0530<br>@@ -4722,7 +4722,14 @@<br>     dest, branches = hg.parseurl(dest, opts.get('branch'))<br>     ui.status(_('pushing to %s\n') % util.hidepassword(dest))<br>
     revs, checkout = hg.addbranchrevs(repo, repo, branches, opts.get('rev'))<br>-    other = hg.peer(repo, opts, dest)<br>+    if dest == "default-push":<br>+        try:<br>+            other = hg.peer(repo, opts, dest)<br>
+        except error.RepoError:<br>+            raise util.Abort(_("repository default-push not found!"),<br>+                         hint=_("see the \"path\" section in \"hg help config\""))<br>
+    else:        <br>+        other = hg.peer(repo, opts, dest)<br>     if revs:<br>         revs = [repo.lookup(r) for r in scmutil.revrange(repo, revs)]<br><br><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, Feb 11, 2014 at 11:33 PM, Matt Mackall <span dir="ltr"><<a href="mailto:mpm@selenic.com" target="_blank">mpm@selenic.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On Wed, 2014-02-12 at 01:24 +0530, Anurag Goel wrote:<br>
> For the above patch, i have fixed the changes which you told me to do so<br>
> and also run the test suits but get some weird result.<br>
><br>
> When I type "make tests" ,it stuck into some loop but when i test each file<br>
> individually, it works fine.<br>
><br>
> My query is how did you get that i hadn't run the test suite and what do<br>
> mean by phrase "just let the Abort command span multiple lines" ?<br>
<br>
</div>The test suite runs check-code, which checks your coding style. That<br>
line contains a coding style error.<br>
<br>
You could have written this as:<br>
<br>
raise util.Abort(_("some long string...."),<br>
                 hint=_("some other long string"))<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Mathematics is the supreme nostalgia of our time.<br>
<br>
<br>
</font></span></blockquote></div><br></div>