Add LC_ALL to implicit tox passenv (#2162)

ssbarnea · web-flow · commit bb609f5acfc5 · 2021-08-18T12:25:47.000+01:00
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
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",

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+include ``LC_ALL`` to implicit list of passenv variables - by :user:`ssbarnea`