18 Aug 14:45
[Cython] Request for review: runtests.py and NumPy issues
From: Dag Sverre Seljebotn <dagss@...>
Subject: [Cython] Request for review: runtests.py and NumPy issues
Newsgroups: gmane.comp.python.cython.devel
Date: 2008-08-18 12:48:01 GMT
Subject: [Cython] Request for review: runtests.py and NumPy issues
Newsgroups: gmane.comp.python.cython.devel
Date: 2008-08-18 12:48:01 GMT
As we're so close to a release I'm going to post a patch here instead of commiting it straight away. The patch fixes an issue with runtests.py with a lot of releases of NumPy. The latest svn version doesn't have the problem but the one shipped with the latest Ubuntu does, so I expect a lot of users may encounter it. The problem is that NumPy doesn't like distutils.unixccompiler being imported already. One could think that the test environment should be further "sandboxed" and that all modules should be removed, but I don't know anything about sys.modules really so I'll leave that to somebody else. Something else: The "cimport" triggers a module import so the numpy testcase fails on systems where numpy is not installed. This isn't all that serious I feel (a failing numpy testcase will communicate the right thing to users not having numpy installed) so I think we should wait until after the release (and then add some kind of "_USES" special testcase variable). -- Dag Sverre
As we're so close to a release I'm going to post a patch here instead of commiting it straight away. The patch fixes an issue with runtests.py with a lot of releases of NumPy. The latest svn version doesn't have the problem but the one shipped with the latest Ubuntu does, so I expect a lot of users may encounter it. The problem is that NumPy doesn't like distutils.unixccompiler being imported already. One could think that the test environment should be further "sandboxed" and that all modules should be removed, but I don't know anything about sys.modules really so I'll leave that to somebody else. Something else: The "cimport" triggers a module import so the numpy testcase fails on systems where numpy is not installed. This isn't all that serious I feel (a failing numpy testcase will communicate the right thing to users not having numpy installed) so I think we should wait until after the release (and then add some kind of "_USES" special testcase variable). -- -- Dag Sverre
RSS Feed