Hi Alexander,

thanks for the patch!  Overall, the change looks good.  I see only a
couple of issues that we have to resolve:

src/platform-freebsd.cc:

  // XXX including this should get __LONG_LONG_SUPPORTED...
  #include <sys/cdefs.h>
  #define __LONG_LONG_SUPPORTED
  #include <sys/ucontext.h>

This should be resolved somehow.

SConstruct:

Why do you need the explicit CCFLAGS, LIBPATH, and LIBS parts on
FreeBSD?  I would have expected this file to be almost untouched.

Thanks,    -- Mads

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to