This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author nikratio
Recipients belopolsky, flxkid, gvanrossum, jafo, mamrhein, nikratio, vdupras
Date 2014-03-16.19:26:57
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1394998018.66.0.102840184378.issue1738@psf.upfronthosting.co.za>
In-reply-to
Content
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.
History
Date User Action Args
2014-03-16 19:26:58nikratiosetrecipients: + nikratio, gvanrossum, jafo, belopolsky, mamrhein, flxkid, vdupras
2014-03-16 19:26:58nikratiosetmessageid: <1394998018.66.0.102840184378.issue1738@psf.upfronthosting.co.za>
2014-03-16 19:26:58nikratiolinkissue1738 messages
2014-03-16 19:26:58nikratiocreate