yuja added a comment.
Looks good, but test-check-config.t failed probably because the doc couldn't be found by the checker. > @@ -464,9 +488,14 @@ > > showstderr(ui, fixctx.rev(), fixername, stderr) > if proc.returncode == 0: > newdata = newerdata > > - elif not stderr: > - showstderr(ui, fixctx.rev(), fixername, > - _('exited with status %d\n') % (proc.returncode,)) + else: + if not stderr: + message = _('exited with status %d\n') % (proc.returncode,) + showstderr(ui, fixctx.rev(), fixername, message) + checktoolfailureaction( + ui, _('no fixes will be applied'), + hint=_('use --config fix.failure=continue to apply any ' + 'successful fixes anyway')) This runs after fork(), but it should work as long as no user interaction is required. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D5200 To: hooper, #hg-reviewers Cc: yuja, mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel