make distutils.core.run_setup re-entrant

[etrepum]

BPO	1051216
Nosy	@etrepum, @tarekziade, @merwok
Files	distutils_run_setup.patch: make distutils.core.run_setup re-entrant

^{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/merwok'
closed_at = <Date 2012-02-11.05:27:45.584>
created_at = <Date 2004-10-21.03:40:09.000>
labels = ['type-bug', 'library']
title = 'make distutils.core.run_setup re-entrant'
updated_at = <Date 2012-02-11.05:27:45.580>
user = 'https://github.com/etrepum'

bugs.python.org fields:

activity = <Date 2012-02-11.05:27:45.580>
actor = 'eric.araujo'
assignee = 'eric.araujo'
closed = True
closed_date = <Date 2012-02-11.05:27:45.584>
closer = 'eric.araujo'
components = ['Distutils']
creation = <Date 2004-10-21.03:40:09.000>
creator = 'bob.ippolito'
dependencies = []
files = ['6314']
hgrepos = []
issue_num = 1051216
keywords = ['patch']
message_count = 4.0
messages = ['47106', '121007', '121008', '153099']
nosy_count = 3.0
nosy_names = ['bob.ippolito', 'tarek', 'eric.araujo']
pr_nums = []
priority = 'normal'
resolution = 'wont fix'
stage = 'test needed'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue1051216'
versions = ['Python 2.7', 'Python 3.2', 'Python 3.3']

[etrepum]

run_setup does not behave as advertised if the called script also
uses run_setup. This patch should make run_setup re-entrant.

[merwok]

Hi Bob, thanks for the report and patch. Could you add a test for this behavior?

(This is irrelevant for distutils2, where run_setup is gone.)

[merwok]

To clarify my position: This does not look like a sane use case, but if the code does not behave as documented it’s a bug that should be fixed. Do you remember how you discovered the bug?

On second thought, I wonder if it’s useful to fix this. Since there is a bug, are people using the function? If not, is it worth fixing, since the function is removed in distutils2?

[merwok]

I’ll reopen this report if someone can provide an example that triggers the bug.