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 eric.araujo
Recipients Adam.Groszer, benjamin.peterson, cjw296, eric.araujo, georg.brandl, jaraco, nadeem.vawda, tarek
Date 2012-02-25.06:41:00
SpamBayes Score 6.8833828e-15
Marked as misclassified No
Message-id <1330152063.07.0.190442341764.issue6884@psf.upfronthosting.co.za>
In-reply-to
Content
> - test_glob_to_re() was doing two levels of escaping (r'\' -> r'\\\\')
>   for its expected output when it should only do one (r'\' -> r'\\').
Fix merged.  I don’t fully understand why one place needs two escapes and the others just one.

> - test_process_template() was not converting some of its expected
>   results to use the native directory separator.
Overlooked paths fixed.

> I also took the liberty of changing the checks for whether the separator needs to be escaped
> - it's better to escape everything except "/", just in case someone decides to port Python to
> some platform using a weird directory separator that is neither "/" nor r"\".
I didn’t take that part.  If someone wants to port Python with a new style of os.sep or os.extsep, I’ll deal with it when they report that.

The buildout.cfg and VCS dirs issues are covered by unit tests for both filelist and sdist, but thanks for testing manually with a real setup.py too.  The third problem you mention (inclusion of unwanted module in a subdir) is also covered by test_filelist.

On a related subject, the code in sdist that excludes VCS dirs uses r'/|\\'; I thought that os.altsep was an alternate *accepted* separator, but not actually produced by OS calls.  IOW, I’m asking if os.walk (used to create filelist.allfiles) can ever give paths with '/' on Windows.  If not, then the regex is redundant, and I can clean it in packaging.  My patch removes the redundancy, just for curiosity, but I won’t actually commit that part to distutils.
History
Date User Action Args
2012-02-25 06:41:03eric.araujosetrecipients: + eric.araujo, georg.brandl, jaraco, cjw296, nadeem.vawda, benjamin.peterson, tarek, Adam.Groszer
2012-02-25 06:41:03eric.araujosetmessageid: <1330152063.07.0.190442341764.issue6884@psf.upfronthosting.co.za>
2012-02-25 06:41:02eric.araujolinkissue6884 messages
2012-02-25 06:41:01eric.araujocreate