scmutil: unify some 'no changes found' messages

This keeps the wording more consistent for secret csets

mercurial/commands.py

@@ -983,6 +983,7 b' def bundle(ui, repo, fname, dest=None, *'
         common = [repo.lookup(rev) for rev in base]
         heads = revs and map(repo.lookup, revs) or revs
         cg = repo.getbundle('bundle', heads=heads, common=common)
+        outgoing = None
     else:
         dest = ui.expandpath(dest or 'default-push', dest or 'default')
         dest, branches = hg.parseurl(dest, opts.get('branch'))
@@ -994,11 +995,7 b' def bundle(ui, repo, fname, dest=None, *'
                                                 force=opts.get('force'))
         cg = repo.getlocalbundle('bundle', outgoing)
     if not cg:
-        if 'outgoing' in locals() and outgoing.excluded:
-            repo.ui.status(_("no changes found but %i secret changesets\n")
-                           % len(outgoing.excluded))
-        else:
-            ui.status(_("no changes found\n"))
+        scmutil.nochangesfound(ui, outgoing and outgoing.excluded)
         return 1
 
     bundletype = opts.get('type', 'bzip2').lower()

mercurial/hg.py

@@ -10,7 +10,7 b' from i18n import _'
 from lock import release
 from node import hex, nullid
 import localrepo, bundlerepo, httprepo, sshrepo, statichttprepo, bookmarks
-import lock, util, extensions, error, node
+import lock, util, extensions, error, node, scmutil
 import cmdutil, discovery
 import merge as mergemod
 import verify as verifymod
@@ -511,11 +511,7 b' def _outgoing(ui, repo, dest, opts):'
                                             force=opts.get('force'))
     o = outgoing.missing
     if not o:
-        if outgoing.excluded:
-            repo.ui.status(_("no outgoing changes but %i secret changesets\n")
-                           % len(outgoing.excluded))
-        else:
-            ui.status(_("no changes found\n"))
+        scmutil.nochangesfound(repo.ui, outgoing.excluded)
         return None
     return o
 

mercurial/localrepo.py

@@ -1629,11 +1629,7 b' class localrepository(repo.repository):'
 
                 if not outgoing.missing:
                     # nothing to push
-                    if outgoing.excluded:
-                        msg = "no changes to push but %i secret changesets\n"
-                        self.ui.status(_(msg) % len(outgoing.excluded))
-                    else:
-                        self.ui.status(_("no changes found\n"))
+                    scmutil.nochangesfound(self.ui, outgoing.excluded)
                     ret = 1
                 else:
                     # something to push

mercurial/scmutil.py

@@ -10,6 +10,14 b' import util, error, osutil, revset, simi'
 import match as matchmod
 import os, errno, re, stat, sys, glob
 
+def nochangesfound(ui, secretlist=None):
+    '''report no changes for push/pull'''
+    if secretlist:
+        ui.status(_("no changes found (ignored %d secret changesets)\n")
+                  % len(secretlist))
+    else:
+        ui.status(_("no changes found\n"))
+
 def checkfilename(f):
     '''Check that the filename f is an acceptable filename for a tracked file'''
     if '\r' in f or '\n' in f:

tests/test-bundle.t

@@ -44,7 +44,7 b' Bundle and phase'
   $ hg -R test phase --force --secret 0
   $ hg -R test bundle phase.hg empty
   searching for changes
-  no changes found but 9 secret changesets
+  no changes found (ignored 9 secret changesets)
   [1]
   $ hg -R test phase --draft -r 'head()'
 

tests/test-incoming-outgoing.t

@@ -405,7 +405,7 b' test outgoing with secret changesets'
   $ hg -R test-dev outgoing test
   comparing with test
   searching for changes
-  no outgoing changes but 5 secret changesets
+  no changes found (ignored 5 secret changesets)
   [1]
   $ hg -R test-dev phase --draft -r 'head()'
 

tests/test-mq-safety.t

@@ -149,7 +149,7 b' All secret'
   $ hg push -r . ../forcepush2
   pushing to ../forcepush2
   searching for changes
-  no changes to push but 1 secret changesets
+  no changes found (ignored 1 secret changesets)
 
 some draft
 
@@ -183,7 +183,7 b' Pushing applied patch with --force'
   $ hg push --force -r default ../forcepush2
   pushing to ../forcepush2
   searching for changes
-  no changes to push but 1 secret changesets
+  no changes found (ignored 1 secret changesets)
   $ hg phase --draft 'mq()'
   $ hg push --force -r default ../forcepush2
   pushing to ../forcepush2