diff hgext/remotefilelog/repack.py @ 43854:612b4b63fb22 stable

remotefilelog: remove the `ensurestart` usage

The option does not seems to do what we want since it does not wait on the background
process actually starting (eg: taking the lock). And the race have been dealt
with another way anyway.

This effectively back out changeset 2c74337e6483.

Differential Revision: https://phab.mercurial-scm.org/D7590

--- a/hgext/remotefilelog/repack.py	Mon Dec 09 11:26:53 2019 +0100
+++ b/hgext/remotefilelog/repack.py	Mon Dec 09 12:29:46 2019 +0100
@@ -36,9 +36,7 @@
     pass
 
 
-def backgroundrepack(
-    repo, incremental=True, packsonly=False, ensurestart=False
-):
+def backgroundrepack(repo, incremental=True, packsonly=False):
     cmd = [procutil.hgexecutable(), b'-R', repo.origroot, b'repack']
     msg = _(b"(running background repack)\n")
     if incremental:
@@ -52,9 +50,7 @@
     if repo.ui.configbool(b'devel', b'remotefilelog.bg-wait'):
         kwargs['record_wait'] = repo.ui.atexit
 
-    procutil.runbgcommand(
-        cmd, encoding.environ, ensurestart=ensurestart, **kwargs
-    )
+    procutil.runbgcommand(cmd, encoding.environ, ensurestart=False, **kwargs)
 
 
 def fullrepack(repo, options=None):