[SciPy-Dev] [PATCH] Scipy and python2.7

Fabian Pedregosa fabian.pedregosa@inria...
Sun Jul 25 07:45:00 CDT 2010


Ralf Gommers wrote:
> Hi Fabian,
> 
> On Thu, Jul 22, 2010 at 8:13 PM, Fabian Pedregosa 
> <fabian.pedregosa@inria.fr <mailto:fabian.pedregosa@inria.fr>> wrote:
> 
>     Hi all.
> 
>     With some minor changes I could install scipy using python2.7. The
>     only thing that failed was the c++ swig wrappers in
>     scipy.sparse.sparsetools. I believe this is because python is
>     compiled now using gcc and not g++, and also default flags have changed.
> 
>     The problem seems to be that numpy/ndarraytypes.h import inttypes.h,
>     but the macros in this file are not accessibly from C++ unless you
>     define the variable __STDC_FORMAT_MACROS. My solution was to define
>     this variable in the setup.py script. This makes it build fine and
>     test are OK too (ubuntu linux 64 bit). I attach the (git) patch.
> 
>  
> That seems to fix the problem, good catch. I'm wondering if this really 
> belongs in setup.py. numpyconfig.h already defines __STDC_FORMAT_MACROS, 
> so maybe including that in sparsetools.i before arrayobject.h would be 
> cleaner?

Thanks for the review. I tried to include numpyconfig.h in sparsetools.i 
as you suggest but for some reason it didn't solve the problem 
(hand-editing the generated swig wrapper and placing that file at start 
made it work, though). Maybe someone with more swig/C/C++ experience 
knows why.

> 
> If it goes in setup.py it should also go in the SConscript file.

I updated the patch in issue #1180.

Cheers,

Fabian

> 
> Cheers,
> Ralf
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> SciPy-Dev mailing list
> SciPy-Dev@scipy.org
> http://mail.scipy.org/mailman/listinfo/scipy-dev



More information about the SciPy-Dev mailing list