summaryrefslogtreecommitdiff
path: root/tools/buildman
diff options
context:
space:
mode:
authorTom Rini <[email protected]>2025-04-15 12:10:26 -0600
committerTom Rini <[email protected]>2025-04-24 15:37:27 -0600
commitefd00b03455644b18eaf2b6dcd07f6d55b95b39e (patch)
tree037d898da2753ccf92134099cc643e51b7b3d38d /tools/buildman
parent10f48365112b164bee6564033ab682747efcb483 (diff)
python: Use and refer to the venv module rather than virtualenv
Using some form of sandbox with Python modules is a long standing best practice with the language. There are a number of ways to have a Python sandbox be created. At this point in time, it seems the Python community is moving towards using the "venv" module provided with Python rather than a separate tool. To match that we make the following changes: - Refer to a "Python sandbox" rather than virtualenv in comments, etc. - Install the python3-venv module in our container and not virtualenv. - In our CI files, invoke "python -m venv" rather than "virtualenv". - In documentation, tell users to install python3-venv and not virtualenv. Signed-off-by: Tom Rini <[email protected]>
Diffstat (limited to 'tools/buildman')
-rw-r--r--tools/buildman/test.py2
-rw-r--r--tools/buildman/toolchain.py2
2 files changed, 2 insertions, 2 deletions
diff --git a/tools/buildman/test.py b/tools/buildman/test.py
index c5feb74a105..e31e6c72e1a 100644
--- a/tools/buildman/test.py
+++ b/tools/buildman/test.py
@@ -987,7 +987,7 @@ class TestBuild(unittest.TestCase):
diff = self.call_make_environment(tchn, full_path=True)[0]
self.assertEqual({b'LC_ALL': b'C'}, diff)
- # Test that virtualenv is handled correctly
+ # Test that Python sandbox is handled correctly
tchn.override_toolchain = False
sys.prefix = '/some/venv'
env = dict(os.environb)
diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
index f4c832be8d3..5e5bb4b0aed 100644
--- a/tools/buildman/toolchain.py
+++ b/tools/buildman/toolchain.py
@@ -217,7 +217,7 @@ class Toolchain:
elif self.cross:
env[b'CROSS_COMPILE'] = tools.to_bytes(wrapper + self.cross)
- # Detect a Python virtualenv and avoid defeating it
+ # Detect a Python sandbox and avoid defeating it
if sys.prefix != sys.base_prefix:
paths = env[b'PATH'].split(b':')
new_paths = []