spectral added a comment.
Someone pointed me at https://bitbucket.org/octobus/mercurial-devel/commits/668806cb43a0 and mentioned discussions on the devel mailing list that I haven't tracked down yet. Theirs is more complete and probably better than mine, so it would be nice to unify them if possible, at least in configuration. Since this patch is only touching 'ui.debug()', it's easy to identify places where we write 'ui.debug(' and don't have a \n['"] in the line, and go and check them to see if they're actually attempting single-line output in multiple invocations. Most of the cases I found were my grep being very simplistic, some I sent patches for in this series because they *should* have had a \n or weren't doing anything, and in the end there was only this one in histedit.py:1189 that I wasn't sure about and didn't touch: ui.debug('histedit: %s is replaced by %s\n' % ( node.short(prec), node.short(succs[0]))) if len(succs) > 1: m = 'histedit: %s' for n in succs[1:]: ui.debug(m % node.short(n)) I'll try to track down those mailing list conversations now REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D1721 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
