[issue1738] filecmp.dircmp does exact match only

Nikolaus Rath report at bugs.python.org
Sun Mar 16 20:26:58 CET 2014


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 <report at bugs.python.org>
<http://bugs.python.org/issue1738>
_______________________________________


More information about the Python-bugs-list mailing list