<br><br><div class="gmail_quote">On Sun, Jul 25, 2010 at 8:45 PM, Fabian Pedregosa <span dir="ltr">&lt;<a href="mailto:fabian.pedregosa@inria.fr" target="_blank">fabian.pedregosa@inria.fr</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div>Ralf Gommers wrote:<br>
&gt; Hi Fabian,<br>
&gt;<br>
&gt; On Thu, Jul 22, 2010 at 8:13 PM, Fabian Pedregosa<br>
</div><div>&gt; &lt;<a href="mailto:fabian.pedregosa@inria.fr" target="_blank">fabian.pedregosa@inria.fr</a> &lt;mailto:<a href="mailto:fabian.pedregosa@inria.fr" target="_blank">fabian.pedregosa@inria.fr</a>&gt;&gt; wrote:<br>

&gt;<br>
&gt;     Hi all.<br>
&gt;<br>
&gt;     With some minor changes I could install scipy using python2.7. The<br>
&gt;     only thing that failed was the c++ swig wrappers in<br>
&gt;     scipy.sparse.sparsetools. I believe this is because python is<br>
&gt;     compiled now using gcc and not g++, and also default flags have changed.<br>
&gt;<br>
&gt;     The problem seems to be that numpy/ndarraytypes.h import inttypes.h,<br>
&gt;     but the macros in this file are not accessibly from C++ unless you<br>
&gt;     define the variable __STDC_FORMAT_MACROS. My solution was to define<br>
&gt;     this variable in the setup.py script. This makes it build fine and<br>
&gt;     test are OK too (ubuntu linux 64 bit). I attach the (git) patch.<br>
&gt;<br>
&gt;<br>
&gt; That seems to fix the problem, good catch. I&#39;m wondering if this really<br>
&gt; belongs in setup.py. numpyconfig.h already defines __STDC_FORMAT_MACROS,<br>
&gt; so maybe including that in sparsetools.i before arrayobject.h would be<br>
&gt; cleaner?<br>
<br>
</div>Thanks for the review. I tried to include numpyconfig.h in sparsetools.i<br>
as you suggest but for some reason it didn&#39;t solve the problem<br>
(hand-editing the generated swig wrapper and placing that file at start<br>
made it work, though). Maybe someone with more swig/C/C++ experience<br>
knows why.<br>
<div> <br></div></blockquote><div>Hmm, no idea. But admittedly I don&#39;t know too much about SWIG or C++.<br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div>
&gt;<br>
&gt; If it goes in setup.py it should also go in the SConscript file.<br>
<br>
</div>I updated the patch in issue #1180.<br>
<br></blockquote><div>Thanks, looks good. This is an important patch, so unless someone objects it will go in in the next few days.<br><br>Cheers,<br>Ralf<br><br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

Cheers,<br>
<br>
Fabian<br><br>
</blockquote></div>