Nadeem Vawda <nadeem.va...@gmail.com> added the comment: There were bugs in two of the updated tests:
- test_glob_to_re() was doing two levels of escaping (r'\' -> r'\\\\') for its expected output when it should only do one (r'\' -> r'\\'). - test_process_template() was not converting some of its expected results to use the native directory separator. I've attached a patch that fixes these issues. I also took the liberty of changing the checks for whether the separator needs to be escaped - it's better to escape everything except "/", just in case someone decides to port Python to some platform using a weird directory separator that is neither "/" nor r"\". With my updated patch, all tests pass on both Windows and Linux, and I've also manually verified that the buildout.cfg problem and the sandbox/dummy.py problem (from issue 9691) are both fixed. Please review (and commit, if my changes are OK). ---------- Added file: http://bugs.python.org/file24627/filelist-regex-v2.diff _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue6884> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
