removed string module use in distutils

[tarekziade]

BPO	3985
Nosy	@tarekziade, @ezio-melotti, @merwok, @akitada
Files	dist-no-string.diff

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/tarekziade'
closed_at = <Date 2010-08-27.21:45:05.132>
created_at = <Date 2008-09-28.10:30:52.710>
labels = ['library']
title = 'removed string module use in distutils'
updated_at = <Date 2010-08-27.21:45:05.129>
user = 'https://github.com/tarekziade'

bugs.python.org fields:

activity = <Date 2010-08-27.21:45:05.129>
actor = 'eric.araujo'
assignee = 'tarek'
closed = True
closed_date = <Date 2010-08-27.21:45:05.132>
closer = 'eric.araujo'
components = ['Distutils', 'Distutils2']
creation = <Date 2008-09-28.10:30:52.710>
creator = 'tarek'
dependencies = []
files = ['11639']
hgrepos = []
issue_num = 3985
keywords = ['patch']
message_count = 6.0
messages = ['73965', '83279', '102676', '102692', '110689', '115134']
nosy_count = 5.0
nosy_names = ['tarek', 'ezio.melotti', 'eric.araujo', 'akitada', 'BreamoreBoy']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue3985'
versions = ['Python 3.1', 'Python 2.7']

[tarekziade]

This patch removes string usage from dist.py, so the module uses modern
syntax.

[akitada]

The patch looks ok to me.
Python 2.3 - 2.6 seem working fine with this patch, too.

[ezio-melotti]

Patch looks good.
There's only a missing space before the for in:
+ self.metadata.platforms = [kw.strip()
+ for kw in platformlist.split(',')]

[tarekziade]

Distutils is now frozen. Moving it to Distutils2

[BreamoreBoy]

Tarek, has this move to distutils2 taken place so we can close this, or do you wish to leave this issue open?

[merwok]

Fixed in r73121 for 2.7 and years ago in 3.0. distutils2 was forked after the fix in 2.7.