Nikolaus Rath added the comment:

I don't think that we can just introduce path normalization in phase0. Even 
though I agree that this would be the proper way to do it when reimplementing 
from scratch, it breaks backward compatibility.

There also is a small mistake in that the *match* attribute should also be used 
for subdirectories in the `phase4` method.

Other than that, this patch looks good to me. I fixed the above issues, rebased 
on current hg tip, and added some missing markup in the documentation. After 
inspecting the code, it seems that there is no difference between directory 
entries being "hidden" by the *hide* parameter, and being "ignored* by the 
*ignore* parameter, so I also updated the documentation make this less 
confusing.

I could not reproduce the test failure reported by Mark, but this is most 
likely because I could not find out on what base revision to apply his patch. 

I think this is ready for commit.

----------
nosy: +nikratio
Added file: http://bugs.python.org/file34444/issue1738.diff

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue1738>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to