[issue1702551] distutils sdist does not exclude SVN/CVS files on Windows

Tarek Ziadé added the comment: Georg, I think I am supposed to be registered since a few days but I don't receive any mail yet. I'll ask... I didn't use a raw string because '\.' is not an escape sequence, so: >>> '\.svn' == '\\.svn' and '\.svn' == r'\.svn' True __

[issue1702551] distutils sdist does not exclude SVN/CVS files on Windows

Georg Brandl added the comment: Tarek, I don't know if you are already subscribed to the python-checkins mailing list -- I've reviewed the commit and posted a reply with a minor problem there. -- nosy: +georg.brandl ___ Python tracker

[issue1702551] distutils sdist does not exclude SVN/CVS files on Windows

Tarek Ziadé added the comment: Fixed in r68276. I have added a test together with the patch, and slighty changed your fix. It's applied in the trunk, and 2.6 as well (it will be forwardported into 3.x as well) Thanks for the feedback and the solution ! -- status: open -> closed

[issue1702551] distutils sdist does not exclude SVN/CVS files on Windows

Tarek Ziadé added the comment: I have put this ticket in my pile. I will write the test to demonstrate the problem and get back to your patch proposal. As Christian said, both separator should be taken care of under Windows, so the final regexp will be slighly different. Last, the trunk code

[issue1702551] distutils sdist does not exclude SVN/CVS files on Windows

Changes by Ralf Schmitt <[EMAIL PROTECTED]>: -- nosy: +schmir _ Tracker <[EMAIL PROTECTED]> _ ___ Python-bugs-list mailing list Unsu

[issue1702551] distutils sdist does not exclude SVN/CVS files on Windows

Christian Heimes added the comment: The code should use \ and / on Windows. -- nosy: +tiran type: -> behavior versions: +Python 2.6 -Python 2.5 _ Tracker <[EMAIL PROTECTED]> _