diff options
author | Gregory P. Smith <greg@krypto.org> | 2023-02-11 20:54:28 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-11 20:54:28 -0800 |
commit | 1d194235e4d5981b5fea25c75318d61189103a58 (patch) | |
tree | ae4eb3ce046118bcafa828044d71ae502387dc02 /Tools | |
parent | Fix typo in test_fstring.py (#101823) (diff) | |
download | cpython-1d194235e4d5981b5fea25c75318d61189103a58.tar.gz cpython-1d194235e4d5981b5fea25c75318d61189103a58.tar.bz2 cpython-1d194235e4d5981b5fea25c75318d61189103a58.zip |
gh-89792: Prevent test_tools from copying 1000M of "source" in freeze test (#101837)
Prevent test_tools from copying 1000M of "source"
It doesn't need a git repo, just the checkout. We skip .git metadata, Doc/build, Doc/venv, and `__pycache__` subdirs, that developers often have in their clients to reduce the size of the source tree copy ten-fold.
This should significantly reduce IO and presumably time on buildbots during this long test.
Diffstat (limited to 'Tools')
-rw-r--r-- | Tools/freeze/test/freeze.py | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/Tools/freeze/test/freeze.py b/Tools/freeze/test/freeze.py index ddbfd7fc9c2..0ae983b15c9 100644 --- a/Tools/freeze/test/freeze.py +++ b/Tools/freeze/test/freeze.py @@ -80,7 +80,19 @@ def copy_source_tree(newroot, oldroot): if newroot == SRCDIR: raise Exception('this probably isn\'t what you wanted') shutil.rmtree(newroot) - shutil.copytree(oldroot, newroot) + + def ignore_non_src(src, names): + """Turns what could be a 1000M copy into a 100M copy.""" + # Don't copy the ~600M+ of needless git repo metadata. + # source only, ignore cached .pyc files. + subdirs_to_skip = {'.git', '__pycache__'} + if os.path.basename(src) == 'Doc': + # Another potential ~250M+ of non test related data. + subdirs_to_skip.add('build') + subdirs_to_skip.add('venv') + return subdirs_to_skip + + shutil.copytree(oldroot, newroot, ignore=ignore_non_src) if os.path.exists(os.path.join(newroot, 'Makefile')): _run_quiet([MAKE, 'clean'], newroot) |