[Python-checkins] bpo-33944: site: Add site-packages tracing in verbose mode (GH-12110)
native-api
webhook-mailer at python.org
Fri Jun 12 02:20:21 EDT 2020
https://github.com/python/cpython/commit/2145c8c9724287a310bc77a2760d4f1c0ca9eb0c
commit: 2145c8c9724287a310bc77a2760d4f1c0ca9eb0c
branch: master
author: native-api <vano at mail.mipt.ru>
committer: GitHub <noreply at github.com>
date: 2020-06-12T15:20:11+09:00
summary:
bpo-33944: site: Add site-packages tracing in verbose mode (GH-12110)
files:
A Misc/NEWS.d/next/Library/2019-03-01-01-56-23.bpo-33944.-82Pkt.rst
M Doc/using/cmdline.rst
M Lib/site.py
M Lib/test/test_site.py
diff --git a/Doc/using/cmdline.rst b/Doc/using/cmdline.rst
index 7aacd8ffe822e..8c65d99ef31f9 100644
--- a/Doc/using/cmdline.rst
+++ b/Doc/using/cmdline.rst
@@ -369,6 +369,11 @@ Miscellaneous options
(filename or built-in module) from which it is loaded. When given twice
(:option:`!-vv`), print a message for each file that is checked for when
searching for a module. Also provides information on module cleanup at exit.
+
+ .. versionchanged:: 3.10
+ The :mod:`site` module reports the site-specific paths
+ and :file:`.pth` files being processed.
+
See also :envvar:`PYTHONVERBOSE`.
diff --git a/Lib/site.py b/Lib/site.py
index e981a142088fd..544306cd40e32 100644
--- a/Lib/site.py
+++ b/Lib/site.py
@@ -88,6 +88,11 @@
USER_BASE = None
+def _trace(message):
+ if sys.flags.verbose:
+ print(message, file=sys.stderr)
+
+
def makepath(*paths):
dir = os.path.join(*paths)
try:
@@ -156,6 +161,7 @@ def addpackage(sitedir, name, known_paths):
else:
reset = False
fullname = os.path.join(sitedir, name)
+ _trace(f"Processing .pth file: {fullname!r}")
try:
f = io.TextIOWrapper(io.open_code(fullname))
except OSError:
@@ -190,6 +196,7 @@ def addpackage(sitedir, name, known_paths):
def addsitedir(sitedir, known_paths=None):
"""Add 'sitedir' argument to sys.path if missing and handle .pth files in
'sitedir'"""
+ _trace(f"Adding directory: {sitedir!r}")
if known_paths is None:
known_paths = _init_pathinfo()
reset = True
@@ -310,6 +317,7 @@ def addusersitepackages(known_paths):
"""
# get the per user site-package path
# this call will also make sure USER_BASE and USER_SITE are set
+ _trace("Processing user site-packages")
user_site = getusersitepackages()
if ENABLE_USER_SITE and os.path.isdir(user_site):
@@ -354,6 +362,7 @@ def getsitepackages(prefixes=None):
def addsitepackages(known_paths, prefixes=None):
"""Add site-packages to sys.path"""
+ _trace("Processing global site-packages")
for sitedir in getsitepackages(prefixes):
if os.path.isdir(sitedir):
addsitedir(sitedir, known_paths)
diff --git a/Lib/test/test_site.py b/Lib/test/test_site.py
index 957e7a41d5466..9f4a8bc64f7ee 100644
--- a/Lib/test/test_site.py
+++ b/Lib/test/test_site.py
@@ -13,6 +13,7 @@
import builtins
import encodings
import glob
+import io
import os
import re
import shutil
@@ -320,6 +321,14 @@ def test_no_home_directory(self):
mock_addsitedir.assert_not_called()
self.assertFalse(known_paths)
+ def test_trace(self):
+ message = "bla-bla-bla"
+ for verbose, out in (True, message + "\n"), (False, ""):
+ with mock.patch('sys.flags', mock.Mock(verbose=verbose)), \
+ mock.patch('sys.stderr', io.StringIO()):
+ site._trace(message)
+ self.assertEqual(sys.stderr.getvalue(), out)
+
class PthFile(object):
"""Helper class for handling testing of .pth files"""
diff --git a/Misc/NEWS.d/next/Library/2019-03-01-01-56-23.bpo-33944.-82Pkt.rst b/Misc/NEWS.d/next/Library/2019-03-01-01-56-23.bpo-33944.-82Pkt.rst
new file mode 100644
index 0000000000000..b0c953dd6752e
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2019-03-01-01-56-23.bpo-33944.-82Pkt.rst
@@ -0,0 +1 @@
+Added site.py site-packages tracing in verbose mode.
More information about the Python-checkins
mailing list