D6539: remotefilelog: tell runbgcommand to not block on child process startup
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Tue Jun 18 14:00:15 UTC 2019
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
More information about the Mercurial-devel
mailing list