Patchwork D6166: fix: make the order of the work queue deterministic

login
register
mail settings
Submitter phabricator
Date March 22, 2019, 2:57 a.m.
Message ID <differential-rev-PHID-DREV-ybmt6yiggva5rdptrtdf-req@phab.mercurial-scm.org>
Download mbox | patch
Permalink /patch/39354/
State Superseded
Headers show

Comments

phabricator - March 22, 2019, 2:57 a.m.
hooper created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  This makes any output generated during the parallel phase of execution stable
  if parallelism is disabled. This helps write tests like that in the future.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D6166

AFFECTED FILES
  hgext/fix.py

CHANGE DETAILS




To: hooper, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/hgext/fix.py b/hgext/fix.py
--- a/hgext/fix.py
+++ b/hgext/fix.py
@@ -280,8 +280,8 @@ 
     for rev in sorted(revstofix):
         fixctx = repo[rev]
         match = scmutil.match(fixctx, pats, opts)
-        for path in pathstofix(ui, repo, pats, opts, match, basectxs[rev],
-                               fixctx):
+        for path in sorted(pathstofix(
+                        ui, repo, pats, opts, match, basectxs[rev], fixctx)):
             fctx = fixctx[path]
             if fctx.islink():
                 continue