Éric Araujo <mer...@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 <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
