durin42 created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers.
REVISION SUMMARY These two invocations will always find a binary because they're re-running hg. As a result, we can skip waiting for the subprocess to start running and save a little bit of wall-time. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D6539 AFFECTED FILES hgext/remotefilelog/repack.py hgext/remotefilelog/shallowrepo.py CHANGE DETAILS diff --git a/hgext/remotefilelog/shallowrepo.py b/hgext/remotefilelog/shallowrepo.py --- a/hgext/remotefilelog/shallowrepo.py +++ b/hgext/remotefilelog/shallowrepo.py @@ -190,7 +190,9 @@ cmd.append('--repack') if revs: cmd += ['-r', revs] - procutil.runbgcommand(cmd, encoding.environ) + # We know this command will find a binary, so don't block + # on it starting. + procutil.runbgcommand(cmd, encoding.environ, ensurestart=False) def prefetch(self, revs, base=None, pats=None, opts=None): """Prefetches all the necessary file revisions for the given revs diff --git a/hgext/remotefilelog/repack.py b/hgext/remotefilelog/repack.py --- a/hgext/remotefilelog/repack.py +++ b/hgext/remotefilelog/repack.py @@ -43,7 +43,8 @@ if packsonly: cmd.append('--packsonly') repo.ui.warn(msg) - procutil.runbgcommand(cmd, encoding.environ) + # We know this command will find a binary, so don't block on it starting. + procutil.runbgcommand(cmd, encoding.environ, ensurestart=False) def fullrepack(repo, options=None): """If ``packsonly`` is True, stores creating only loose objects are skipped. To: durin42, #hg-reviewers Cc: mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel