parse_config_file broken on Python 3 #702

Closed
wants to merge 1 commit into
from

2 participants

@whardier

Need to replace the call to execfile().

@bdarnell
tornadoweb member

Tornado 3.0 no longer requires 2to3.

@bdarnell bdarnell closed this Mar 27, 2013
@whardier

So is execfile overridden somewhere then? It is used in options.py and breaks reading config files.

@bdarnell
tornadoweb member

Ah, didn't notice that. Config files are simply broken on python 3 then.

@bdarnell bdarnell reopened this Mar 27, 2013
@bdarnell bdarnell added a commit that closed this pull request Mar 28, 2013
@bdarnell bdarnell Fix options.parse_config_file on Python 3.
Add a test for this function.

Closes #702.
dd388bf
@bdarnell bdarnell closed this in dd388bf Mar 28, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment