Dag Sverre Seljebotn | 18 Aug 14:45

[Cython] Request for review: runtests.py and NumPy issues

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

Gmane