On May 15, 2010, at 20:03 , Jason Grout wrote:

In fixing #8756, I've upgraded the graph planarity code to the most recent rewrite of John Boyer's planarity code. Unfortunately, the C code has several instances of:

#include <malloc.h>

I just found out that OSX does not like this, where it should apparently either be

#include <malloc/malloc.h>

(or maybe there's no need to even include malloc.h at all.)

This is the right answer.

Surely this is not the first time this problem has been run into among this crowd. How is this typically dealt with when compiling a C file? Should I make a new include directory in module_list.py (where the appropriate .pyx files are compiled with the C file), or modify the C code to detect OSX, or something else?

I generally do the second: detect Mac OS X and drop the include.

HTH

Justin

--
Justin C. Walker, Curmudgeon-At-Large, Director
Institute for the Enhancement of the Director's Income
--------
The path of least resistance:
it's not just for electricity any more.
--------



--
To post to this group, send an email to sage-devel@googlegroups.com
To unsubscribe from this group, send an email to 
sage-devel+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org

Reply via email to