# HG changeset patch # User Augie Fackler # Date 1419287251 18000 # Node ID 2205d00b6d2b0d8abefd5034337f7eb58a970710 # Parent 70afc58c32d3619ca2161eb3fc9d6d30a57e53f7 demandimport: blacklist distutils.msvc9compiler (issue4475) This module depends on _winreg, which is windows-only. Recent versions of setuptools load distutils.msvc9compiler and expect it to ImportError immediately when on non-Windows platforms, so we need to let them do that. This breaks in an especially mystifying way, because setuptools uses vars() on the imported module. We then throw an exception, which vars doesn't pick up on well. For example: In [3]: class wat(object): ...: @property ...: def __dict__(self): ...: assert False ...: In [4]: vars(wat()) --------------------------------------------------------------------------- TypeError Traceback (most recent call last) in () ----> 1 vars(wat()) TypeError: vars() argument must have __dict__ attribute Which is similar to the problem we run into. diff -r 70afc58c32d3 -r 2205d00b6d2b mercurial/demandimport.py --- a/mercurial/demandimport.py Sun Dec 21 15:06:54 2014 -0500 +++ b/mercurial/demandimport.py Mon Dec 22 17:27:31 2014 -0500 @@ -164,6 +164,8 @@ '_ssl', # conditional imports in the stdlib, issue1964 'rfc822', 'mimetools', + # setuptools 8 expects this module to explode early when not on windows + 'distutils.msvc9compiler' ] def isenabled(): diff -r 70afc58c32d3 -r 2205d00b6d2b tests/test-demandimport.py --- a/tests/test-demandimport.py Sun Dec 21 15:06:54 2014 -0500 +++ b/tests/test-demandimport.py Mon Dec 22 17:27:31 2014 -0500 @@ -1,6 +1,16 @@ from mercurial import demandimport demandimport.enable() +import os +if os.name != 'nt': + try: + import distutils.msvc9compiler + print ('distutils.msvc9compiler needs to be an immediate ' + 'importerror on non-windows platforms') + distutils.msvc9compiler + except ImportError: + pass + import re rsub = re.sub