[PATCH] distutils.sysconfig.get_python_lib prefix argument broken

[pjenvey]

BPO	3386
Nosy	@pjenvey, @tarekziade
Files	get_python_lib-r65033.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 2009-02-10.12:42:43.472>
created_at = <Date 2008-07-16.21:10:53.122>
labels = ['type-bug', 'library']
title = '[PATCH] distutils.sysconfig.get_python_lib prefix argument broken'
updated_at = <Date 2009-02-10.12:42:43.471>
user = 'https://github.com/pjenvey'

bugs.python.org fields:

activity = <Date 2009-02-10.12:42:43.471>
actor = 'tarek'
assignee = 'tarek'
closed = True
closed_date = <Date 2009-02-10.12:42:43.472>
closer = 'tarek'
components = ['Distutils']
creation = <Date 2008-07-16.21:10:53.122>
creator = 'pjenvey'
dependencies = []
files = ['10917']
hgrepos = []
issue_num = 3386
keywords = ['patch']
message_count = 3.0
messages = ['69836', '81251', '81543']
nosy_count = 2.0
nosy_names = ['pjenvey', 'tarek']
pr_nums = []
priority = 'high'
resolution = None
stage = None
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue3386'
versions = ['Python 2.6', 'Python 3.0', 'Python 3.1', 'Python 2.7']

[pjenvey]

get_python_lib supports an optional prefix argument:

If 'prefix' is supplied, use it instead of sys.prefix or
sys.exec_prefix -- i.e., ignore 'plat_specific'.

However the NT and OS2 platforms don't use the prefix argument when
specified.

This problem was brought up a while ago here: http://mail.python.org/pipermail/distutils-sig/2002-November/003099.html

Andrew (the OS2 maintainer) claimed in the thread that fixing this would
break OS2, but I don't see how. All callers of get_python_lib in the
stdlib don't specify a prefix anyway. Anyone calling it with a prefix
and expecting it not to be used is broken.

[tarekziade]

The patch looks fine to me. I'll send a mail to Andrew to ask him for a
demonstration, and wait a week to commit it.

[tarekziade]

Done in r69485. Thanks for the patch Phillip !