# HG changeset patch
# User Sushil khanchi <sushilkhanchi97@gmail.com>
# Date 1530214508 -19800
# Node ID 999e5c218daf300f8f217b6b5e3df0e7c4ef3511
# Parent  c892a30bafb9e991202f6b4b96b31132f34aceb6
rebase: suppress warning thrown when aborting rebase in case of dryrun

Before this patch dryrun output contained "rebase aborted" every
time we run rebase in dryrun mode and this warning does not sound
safe from a user prespective.

Differential Revision: https://phab.mercurial-scm.org/D3857

diff -r c892a30bafb9 -r 999e5c218daf hgext/rebase.py
--- a/hgext/rebase.py	Fri Jun 29 00:47:33 2018 +0530
+++ b/hgext/rebase.py	Fri Jun 29 01:05:08 2018 +0530
@@ -325,7 +325,7 @@
         skippedset.update(obsoleteextinctsuccessors)
         _checkobsrebase(self.repo, self.ui, obsoleteset, skippedset)
 
-    def _prepareabortorcontinue(self, isabort, backup=True):
+    def _prepareabortorcontinue(self, isabort, backup=True, suppwarns=False):
         try:
             self.restorestatus()
             self.collapsemsg = restorecollapsemsg(self.repo, isabort)
@@ -342,7 +342,8 @@
                 raise error.Abort(msg, hint=hint)
         if isabort:
             return abort(self.repo, self.originalwd, self.destmap, self.state,
-                         activebookmark=self.activebookmark, backup=backup)
+                         activebookmark=self.activebookmark, backup=backup,
+                         suppwarns=suppwarns)
 
     def _preparenewrebase(self, destmap):
         if not destmap:
@@ -851,7 +852,8 @@
             return 0
         finally:
             # no need to store backup in case of dryrun
-            rbsrt._prepareabortorcontinue(isabort=True, backup=False)
+            rbsrt._prepareabortorcontinue(isabort=True, backup=False,
+                                          suppwarns=True)
 
 def _dorebase(ui, repo, inmemory=False, **opts):
     rbsrt = rebaseruntime(repo, ui, inmemory, pycompat.byteskwargs(opts))
@@ -1554,7 +1556,8 @@
 
     return False
 
-def abort(repo, originalwd, destmap, state, activebookmark=None, backup=True):
+def abort(repo, originalwd, destmap, state, activebookmark=None, backup=True,
+          suppwarns=False):
     '''Restore the repository to its original state.  Additional args:
 
     activebookmark: the name of the bookmark that should be active after the
@@ -1607,7 +1610,8 @@
     finally:
         clearstatus(repo)
         clearcollapsemsg(repo)
-        repo.ui.warn(_('rebase aborted\n'))
+        if not suppwarns:
+            repo.ui.warn(_('rebase aborted\n'))
     return 0
 
 def sortsource(destmap):
diff -r c892a30bafb9 -r 999e5c218daf tests/test-rebase-inmemory.t
--- a/tests/test-rebase-inmemory.t	Fri Jun 29 00:47:33 2018 +0530
+++ b/tests/test-rebase-inmemory.t	Fri Jun 29 01:05:08 2018 +0530
@@ -212,7 +212,6 @@
   rebasing 3:055a42cdd887 "d"
   rebasing 4:e860deea161a "e"
   there will be no conflict, you can rebase
-  rebase aborted
 
   $ hg diff
   $ hg status
@@ -245,7 +244,6 @@
   rebasing 3:055a42cdd887 "d"
   rebasing 4:e860deea161a "e"
   there will be no conflict, you can rebase
-  rebase aborted
 
 Check dryrun gives correct results when there is conflict in rebasing
 Make a conflict:
@@ -285,7 +283,6 @@
   transaction abort!
   rollback completed
   hit a merge conflict
-  rebase aborted
   [1]
   $ hg diff
   $ hg status
@@ -321,5 +318,4 @@
   rebasing 4:e860deea161a "e"
   merging e
   hit a merge conflict
-  rebase aborted
   [1]