Patchwork D1597: tests: write and use a custom helper script to avoid find's -printf

login
register
mail settings
Submitter phabricator
Date Dec. 5, 2017, 11:28 p.m.
Message ID <372cc1ef600ed01efe01a6b427a38dcf@localhost.localdomain>
Download mbox | patch
Permalink /patch/25958/
State Not Applicable
Headers show

Comments

phabricator - Dec. 5, 2017, 11:28 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG3d172ca55d3b: tests: write and use a custom helper script to avoid find&#039;s -printf (authored by durin42, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D1597?vs=4134&id=4147

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

AFFECTED FILES
  tests/list-tree.py
  tests/test-largefiles-misc.t

CHANGE DETAILS




To: durin42, #hg-reviewers, martinvonz
Cc: mercurial-devel

Patch

diff --git a/tests/test-largefiles-misc.t b/tests/test-largefiles-misc.t
--- a/tests/test-largefiles-misc.t
+++ b/tests/test-largefiles-misc.t
@@ -1095,8 +1095,7 @@ 
 Move (and then undo) a directory move with only largefiles.
 
   $ listtree() {
-  >   find $@ \( -type d -printf "%p/\n" -o -type f -printf "%p\n" \) \
-  >           -a -name .hg -prune | sort
+  >   $PYTHON $TESTDIR/list-tree.py $@
   > }
 
   $ cd subrepo-root
diff --git a/tests/list-tree.py b/tests/list-tree.py
new file mode 100644
--- /dev/null
+++ b/tests/list-tree.py
@@ -0,0 +1,27 @@ 
+from __future__ import (
+    absolute_import,
+    print_function,
+)
+
+import argparse
+import os
+
+ap = argparse.ArgumentParser()
+ap.add_argument('path', nargs='+')
+opts = ap.parse_args()
+
+def gather():
+    for p in opts.path:
+        if not os.path.exists(p):
+            return
+        if os.path.isdir(p):
+            yield p + os.path.sep
+            for dirpath, dirs, files in os.walk(p):
+                for d in dirs:
+                    yield os.path.join(dirpath, d) + os.path.sep
+                for f in files:
+                    yield os.path.join(dirpath, f)
+        else:
+            yield p
+
+print('\n'.join(sorted(gather())))