D1720: debug: add newlines at the end of three locations that appear to need it

[spectral (Kyle Lippincott)]

This revision was automatically updated to reflect the committed changes.
Closed by commit rHG7906354cbc68: debug: add newlines at the end of three 
locations that appear to need it (authored by spectral, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D1720?vs=4536=4574

REVISION DETAIL
  https://phab.mercurial-scm.org/D1720

AFFECTED FILES
  hgext/patchbomb.py
  mercurial/bundle2.py
  mercurial/dispatch.py

CHANGE DETAILS

diff --git a/mercurial/dispatch.py b/mercurial/dispatch.py
--- a/mercurial/dispatch.py
+++ b/mercurial/dispatch.py
@@ -451,7 +451,7 @@
 return m.group()
 else:
 ui.debug("No argument found for substitution "
- "of %i variable in alias '%s' definition."
+ "of %i variable in alias '%s' definition.\n"
  % (int(m.groups()[0]), self.name))
 return ''
 cmd = re.sub(br'\$(\d+|\$)', _checkvar, self.definition[1:])
diff --git a/mercurial/bundle2.py b/mercurial/bundle2.py
--- a/mercurial/bundle2.py
+++ b/mercurial/bundle2.py
@@ -2056,7 +2056,7 @@
 # The mergemarkers call will crash if marker creation is not enabled.
 # we want to avoid this if the part is advisory.
 if not inpart.mandatory and op.repo.obsstore.readonly:
-op.repo.ui.debug('ignoring obsolescence markers, feature not enabled')
+op.repo.ui.debug('ignoring obsolescence markers, feature not 
enabled\n')
 return
 new = op.repo.obsstore.mergemarkers(tr, markerdata)
 op.repo.invalidatevolatilesets()
diff --git a/hgext/patchbomb.py b/hgext/patchbomb.py
--- a/hgext/patchbomb.py
+++ b/hgext/patchbomb.py
@@ -632,15 +632,15 @@
 # check if revision exist on the public destination
 publicurl = repo.ui.config('patchbomb', 'publicurl')
 if publicurl:
-repo.ui.debug('checking that revision exist in the public repo')
+repo.ui.debug('checking that revision exist in the public repo\n')
 try:
 publicpeer = hg.peer(repo, {}, publicurl)
 except error.RepoError:
 repo.ui.write_err(_('unable to access public repo: %s\n')
   % publicurl)
 raise
 if not publicpeer.capable('known'):
-repo.ui.debug('skipping existence checks: public repo too old')
+repo.ui.debug('skipping existence checks: public repo too old\n')
 else:
 out = [repo[r] for r in revs]
 known = publicpeer.known(h.node() for h in out)



To: spectral, #hg-reviewers, krbullock
Cc: krbullock, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

D1720: debug: add newlines at the end of three locations that appear to need it

[krbullock (Kevin Bullock)]

krbullock accepted this revision.
krbullock added a comment.
This revision is now accepted and ready to land.


  Sure, queued.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D1720

To: spectral, #hg-reviewers, krbullock
Cc: krbullock, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

D1720: debug: add newlines at the end of three locations that appear to need it

[spectral (Kyle Lippincott)]

spectral created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D1720

AFFECTED FILES
  hgext/patchbomb.py
  mercurial/bundle2.py
  mercurial/dispatch.py

CHANGE DETAILS

diff --git a/mercurial/dispatch.py b/mercurial/dispatch.py
--- a/mercurial/dispatch.py
+++ b/mercurial/dispatch.py
@@ -451,7 +451,7 @@
 return m.group()
 else:
 ui.debug("No argument found for substitution "
- "of %i variable in alias '%s' definition."
+ "of %i variable in alias '%s' definition.\n"
  % (int(m.groups()[0]), self.name))
 return ''
 cmd = re.sub(br'\$(\d+|\$)', _checkvar, self.definition[1:])
diff --git a/mercurial/bundle2.py b/mercurial/bundle2.py
--- a/mercurial/bundle2.py
+++ b/mercurial/bundle2.py
@@ -2056,7 +2056,7 @@
 # The mergemarkers call will crash if marker creation is not enabled.
 # we want to avoid this if the part is advisory.
 if not inpart.mandatory and op.repo.obsstore.readonly:
-op.repo.ui.debug('ignoring obsolescence markers, feature not enabled')
+op.repo.ui.debug('ignoring obsolescence markers, feature not 
enabled\n')
 return
 new = op.repo.obsstore.mergemarkers(tr, markerdata)
 op.repo.invalidatevolatilesets()
diff --git a/hgext/patchbomb.py b/hgext/patchbomb.py
--- a/hgext/patchbomb.py
+++ b/hgext/patchbomb.py
@@ -632,15 +632,15 @@
 # check if revision exist on the public destination
 publicurl = repo.ui.config('patchbomb', 'publicurl')
 if publicurl:
-repo.ui.debug('checking that revision exist in the public repo')
+repo.ui.debug('checking that revision exist in the public repo\n')
 try:
 publicpeer = hg.peer(repo, {}, publicurl)
 except error.RepoError:
 repo.ui.write_err(_('unable to access public repo: %s\n')
   % publicurl)
 raise
 if not publicpeer.capable('known'):
-repo.ui.debug('skipping existence checks: public repo too old')
+repo.ui.debug('skipping existence checks: public repo too old\n')
 else:
 out = [repo[r] for r in revs]
 known = publicpeer.known(h.node() for h in out)



To: spectral, #hg-reviewers
Cc: mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel