diff options
| author | Tom Rini <[email protected]> | 2025-04-15 12:10:26 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-04-24 15:37:27 -0600 |
| commit | efd00b03455644b18eaf2b6dcd07f6d55b95b39e (patch) | |
| tree | 037d898da2753ccf92134099cc643e51b7b3d38d /tools/buildman | |
| parent | 10f48365112b164bee6564033ab682747efcb483 (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.py | 2 | ||||
| -rw-r--r-- | tools/buildman/toolchain.py | 2 |
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 = [] |
