Message154119
| Author |
eric.araujo |
| Recipients |
Adam.Groszer, cjw296, eric.araujo, jaraco, nadeem.vawda, tarek |
| Date |
2012年02月24日.07:22:25 |
| SpamBayes Score |
5.04366e-05 |
| Marked as misclassified |
No |
| Message-id |
<1330068148.54.0.419872128605.issue6884@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
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. |
|