Recursive replacement when getdict('setenv')

[pytoxbot]

• Bitbucket: https://bitbucket.org/hpk42/tox/issue/288
• Originally reported by: @kozhukalov
• Originally created at: 2015-11-20T08:57:36.569

The thing is that when config.py module replaces
macros in setenv dict, it calls getdict('setenv')
which recursively tries to replace macros.
So, python reports 'maximum recursion depth exceeded' error.

This patch fixes the issue

--- a/tox/config.py Wed Nov 11 15:58:33 2015 +0100
+++ b/tox/config.py Fri Nov 20 10:45:40 2015 +0300
@@ -835,8 +835,8 @@
             return []
         return [x.strip() for x in s.split(sep) if x.strip()]

-    def getdict(self, name, default=None, sep="\n"):
-        s = self.getstring(name, None)
+    def getdict(self, name, default=None, sep="\n", replace=True):
+        s = self.getstring(name, None, replace=replace)
         if s is None:
             return default or {}

@@ -910,7 +910,7 @@
         return '\n'.join(filter(None, map(factor_line, lines)))

     def _replace_env(self, match):
-        env_list = self.getdict('setenv')
+        env_list = self.getdict('setenv', replace=False)
         match_value = match.group('substitution_value')
         if not match_value:
             raise tox.exception.ConfigError(

[pytoxbot]

Original comment by @kozhukalov

Duplicate of #285.