On Sep 18, 2012, at 07:37 PM, Dmitry Shachnev wrote:

>On Tue, 18/09/2012 11:05 -0400, Barry Warsaw wrote:
>> So I guess if no one else has any other suggestions, then the following
>> changes should be made to the patch, after which it could be applied:
>> 
>> * compare against site.getsitepackages() for directories that should not be
>>   removed.
>> * update the docstring to indicate that site package directories are not
>>   removed even if magic_tag is None
>
>Done, the updated patch attached. Thanks for the suggestions!
>
>Also, I've updated my branch at lp:~mitya57/+junk/python3-defaults so
>that it can be easily merged.

I'll commit this to alioth bzr after I do some additional local testing, but
it generally looks pretty good.  Thanks!  I made a few minor changes, such as
using os.path.splitext() instead of assuming the extension is 3 characters
long (it probably always will be, but splitext() is safer).  I also added a
bunch of comments so that it's clearer what's going on.

I'll have to request sponsorship since I can't upload this myself.

Attachment: signature.asc
Description: PGP signature

Reply via email to