94f4bd9c8b
This check turns out to be fairly expensive on Windows (~150ms), and having it at import time means we're taking the hit every time we launch memory_cache_http_server. We only need to do this if the Python files have changed, so move it to runhooks. The reason this check exists in the first place is that we've had problems in the past where e.g. someone renames foo.py to foo/__init__.py. foo.pyc remains on the filesystem of every Chromium checkout and causes a name conflict in Python imports - foo can refer to the compiled module or the package. BUG=388256 TEST=git rm a file in src/tools that has a .pyc; git commit; gclient runhooks; ensure .pyc doesn't exist; undo commit Review URL: https://codereview.chromium.org/489693002 Cr-Commit-Position: refs/heads/master@{#291825}
40 lines
1.1 KiB
Python
Executable File
40 lines
1.1 KiB
Python
Executable File
#!/usr/bin/env python
|
|
# Copyright 2014 The Chromium Authors. All rights reserved.
|
|
# Use of this source code is governed by a BSD-style license that can be
|
|
# found in the LICENSE file.
|
|
|
|
import os
|
|
import sys
|
|
|
|
|
|
def RemoveAllStalePycFiles(base_dir):
|
|
"""Scan directories for old .pyc files without a .py file and delete them."""
|
|
for dirname, _, filenames in os.walk(base_dir):
|
|
if '.svn' in dirname or '.git' in dirname:
|
|
continue
|
|
for filename in filenames:
|
|
root, ext = os.path.splitext(filename)
|
|
if ext != '.pyc':
|
|
continue
|
|
|
|
pyc_path = os.path.join(dirname, filename)
|
|
py_path = os.path.join(dirname, root + '.py')
|
|
|
|
try:
|
|
if not os.path.exists(py_path):
|
|
os.remove(pyc_path)
|
|
except OSError:
|
|
# Wrap OS calls in try/except in case another process touched this file.
|
|
pass
|
|
|
|
try:
|
|
os.removedirs(dirname)
|
|
except OSError:
|
|
# Wrap OS calls in try/except in case another process touched this dir.
|
|
pass
|
|
|
|
|
|
if __name__ == '__main__':
|
|
for path in sys.argv[1:]:
|
|
RemoveAllStalePycFiles(path)
|