[issue23114] "dist must be a Distribution instance" check fails with setuptools

2016-02-13 Thread Ralf Gommers 

Changes by Ralf Gommers :


--
nosy: +ralf.gommers

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23114] dist must be a Distribution instance check fails with setuptools

2014-12-26 Thread Stefan Behnel 

New submission from Stefan Behnel:

distutils uses this code to validate the input in Command.__init__():

# late import because of mutual dependence between these classes
from distutils.dist import Distribution

if not isinstance(dist, Distribution):
raise TypeError, dist must be a Distribution instance

(Lib/distutils/cmd.py, after line 50)

This fails if the calling code uses the stdlib Distribution type (e.g. with a 
module global import) but setuptools was only imported afterwards and replaced 
distutils.dist.Distribution with its own implementation. In this case, the 
above code will get hold of the setuptools implementation and the isinstance() 
check will fail as both classes are not compatible.

I guess this is a problem more in setuptools than in distutils, but a) this 
isinstance() check has a general code smell and b) pip and setuptools are 
automatically installed in a specific (impacted) version in Py2.7.9 and Py3.4, 
which makes this problem quite pressing for the stdlib side.

--
components: Distutils
messages: 233104
nosy: dstufft, eric.araujo, scoder
priority: normal
severity: normal
status: open
title: dist must be a Distribution instance check fails with setuptools
type: behavior
versions: Python 2.7, Python 3.4

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23114
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23114] dist must be a Distribution instance check fails with setuptools

2014-12-26 Thread Éric Araujo 

Éric Araujo added the comment:

Can this ticket be merged into #23102 ?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23114
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23114] dist must be a Distribution instance check fails with setuptools

2014-12-26 Thread Stefan Behnel 

Stefan Behnel added the comment:

Yes, pretty much the same problem.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23114
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23114] dist must be a Distribution instance check fails with setuptools

2014-12-26 Thread Éric Araujo 

Changes by Éric Araujo mer...@netwok.org:


--
resolution:  - duplicate
stage:  - resolved
status: open - closed
superseder:  - distutils: tip-toe around quirks owing to setuptools 
monkey-patching Extension

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23114
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com