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

[Éric Araujo]

Éric Araujo <merwok at netwok.org> added the comment:

Thanks for testing and confirming that this code never worked.  I am now confident that we’re about to fix once and for all this basket of bugs in the correct way.

I updated the tests to make them hopefully pass on Windows, and while doing that I realized that I don’t like the unnecessary escaping of / on posix systems.  Regexes already look like grit on my monitor, so I prefer not making them harder to read.  My latest patch only escapes if os.sep is the backslash.

Should apply cleanly to 2.7, please test.


> While we're talking about translate_pattern(), I have a question about this line:
> [snip]
> For example, it seems wrong that "recursive-include foo bar.*" matches foo/test_bar.py.
Certainly.  Feel free to open another report for that, or I’ll do it.

----------
resolution: accepted -> 
Added file: http://bugs.python.org/file24623/filelist-regex-bugs.diff

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