[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

Nadeem Vawda report at bugs.python.org
Sat Feb 25 09:10:23 CET 2012


Nadeem Vawda <nadeem.vawda at gmail.com> added the comment:

>> - test_glob_to_re() was doing two levels of escaping (r'\' -> r'\\\\')
>>   for its expected output when it should only do one (r'\' -> r'\\').
> Fix merged.  I don’t fully understand why one place needs two escapes and the others just one.

I agree that this code is confusing, though. Perhaps we should add a
comment in glob_to_re():

    # Warning - manipulating a regex with a regex. Here be dragons.

;-)

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue6884>
_______________________________________


More information about the Python-bugs-list mailing list