homepage

+ Bug is specific to Windows platform
+ When creating a source distribution with :
**setup.py sdist**
+ the files in CVS/RCS/.svn folders should be excluded, as specified by paragraph 4.1 of distutils documentation
+ on windows, they are not excluded, because :
 + the pattern is r'/(RCS|CVS|\.svn)/.*'
 + the filepathes are in the perverse Windows style (separated by backslashes)
+ I could exclude those files by doing either :
 + add a <<global-exclude .svn/*>> in manifest.in
 + modify line 357 of distutils/command/sdist.py
+ initial line
self.filelist.exclude_pattern(r'/(RCS|CVS|\.svn)/.*', is_regex=1)
+ modified lines
reossep = (os.sep == '/' and '/' or r'\\')
self.filelist.exclude_pattern(r'%s(RCS|CVS|\.svn)%s.*' % (reossep, reossep), is_regex=1)

The code should use \ and / on Windows.

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 has evolved a bit since your initial proposal, and
now includes other VCSs like Mercurial or Git.

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 !

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.

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