Add LC_ALL to implicit tox passenv

This is needed because LANG by itself is not enough to help newer pip from failing on py36. Forcing tox users to manually alter not only their system configuration but also their tox.ini files to avoid that bug is not quite desired. As we already pass `LANG` and `LANGUAGE` I see not reason why we should not do the same for `LC_ALL` too. Related: pypa/pip#10219
tox-dev · Aug 18, 2021 · 6a51154 · 6a51154
1 parent 8ded7b7
commit 6a51154
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/docs/changelog/2162.bugfix.rst b/docs/changelog/2162.bugfix.rst
@@ -0,0 +1 @@
+include ``LC_ALL`` to implicit list of passenv variables - by :user:`ssbarnea`
diff --git a/src/tox/config/__init__.py b/src/tox/config/__init__.py
@@ -770,6 +770,7 @@ def passenv(testenv_config, value):
             "CURL_CA_BUNDLE",
             "LANG",
             "LANGUAGE",
+            "LC_ALL",
             "LD_LIBRARY_PATH",
             "PATH",
             "PIP_INDEX_URL",