[issue20754] Distribution.parse_config_files uses interpolation since Python 3

2017-07-26 Thread Jason R. Coombs
Jason R. Coombs added the comment: While investigating https://github.com/pypa/setuptools/issues/1062, I discovered that the implementation in the attached patch is insufficient. The 'parse_config_files' calls ConfigParser.__init__ in two places, before the for loop and at the end of each loop

[issue20754] Distribution.parse_config_files uses interpolation since Python 3

2016-12-14 Thread Jason R. Coombs
Jason R. Coombs added the comment: I've created a backport implementation in https://github.com/pypa/setuptools/issues/889. All that remains then is to have a test for Python, which can probably borrow from the fix for issue20120. I'll leave it to others to draft the patch for the test.

[issue20754] Distribution.parse_config_files uses interpolation since Python 3

2016-12-14 Thread Jason R. Coombs
Jason R. Coombs added the comment: I reviewed the docs for configparser, and I agree - using ConfigParser(interpolation=None) is preferable to the "legacy" RawConfigParser. -- ___ Python tracker __

[issue20754] Distribution.parse_config_files uses interpolation since Python 3

2016-12-12 Thread Jason R. Coombs
Jason R. Coombs added the comment: I forget if there was a reason for choosing RawConfigParser over ConfigParaer with interpolation=None. I'd like to know that before choosing the latter. I'd also like to see if appropriate the implementation patched in Setuptools, providing compatibility for

[issue20754] Distribution.parse_config_files uses interpolation since Python 3

2016-12-12 Thread Berker Peksag
Berker Peksag added the comment: Thanks! We also need a test case for the new behavior. -- stage: resolved -> patch review ___ Python tracker ___

[issue20754] Distribution.parse_config_files uses interpolation since Python 3

2016-12-12 Thread Axel Haustant
Axel Haustant added the comment: I just attached the patch submitted on issue28935. The fix is to simply give interpolation=None as ConfigParser parameter (as documented in the official Python 3.x ConfigParser documentation) -- keywords: +patch nosy: +noirbizarre Added file: http://bu

[issue20754] Distribution.parse_config_files uses interpolation since Python 3

2016-12-12 Thread Jason R. Coombs
Jason R. Coombs added the comment: Looking more closely now and as a result of the report in issue28935, I see that this issue is another separate manifestation of the issue addressed in issue20120. -- resolution: out of date -> status: closed -> open title: distutils should use SafeC