Compilation error due to V8 changes

[GoogleCodeExporter]

While compiling PyV8 I got the following error which seems to be due to a 
change in the v8::internal::ParserApi::Parse prototype.

src/Engine.cpp: In member function ‘void 
CScript::visit(boost::python::api::object) const’:
src/Engine.cpp:549:34: error: no matching function for call to 
‘v8::internal::ParserApi::Parse(v8::internal::CompilationInfo*)’
/home/buffer/v8/src/parser.h:172:15: note: candidate is: static bool 
v8::internal::ParserApi::Parse(v8::internal::CompilationInfo*, int)
error: command 'x86_64-pc-linux-gnu-g++' failed with exit status 1

Additional information

buffer@alnitak ~/pyv8 $ svn info
Path: .
URL: http://pyv8.googlecode.com/svn/trunk
Repository Root: http://pyv8.googlecode.com/svn
Repository UUID: 3b770cb8-f9e6-11dd-ac5b-dfceb64ead24
Revision: 425
Node Kind: directory
Schedule: normal
Last Changed Author: flier.lu@gmail.com
Last Changed Rev: 425
Last Changed Date: 2011-11-18 14:13:33 +0100 (Fri, 18 Nov 2011)

buffer@alnitak ~/v8 $ svn info
Path: .
URL: http://v8.googlecode.com/svn/trunk
Repository Root: http://v8.googlecode.com/svn
Repository UUID: ce2b1a6d-e550-0410-aec6-3dcde31c8c00
Revision: 10093
Node Kind: directory
Schedule: normal
Last Changed Author: mstarzinger@chromium.org
Last Changed Rev: 10089
Last Changed Date: 2011-11-29 15:28:56 +0100 (Tue, 29 Nov 2011)

Original issue reported on code.google.com by angelo.d...@gmail.com on 30 Nov 2011 at 11:29

[GoogleCodeExporter]

Attached patch fixes problem with compilation against new v8. Please test.

Original comment by scroo...@gmail.com on 30 Nov 2011 at 3:59

Attachments:

• parse_fix.patch

[GoogleCodeExporter]

Fixed, thanks for your patch, SVN r426

Original comment by flier...@gmail.com on 1 Dec 2011 at 5:23

• Changed state: Fixed
• Added labels: OpSys-All

[GoogleCodeExporter]

I confirm the patch properly fixed the issue. Thanks.

Original comment by angelo.d...@gmail.com on 1 Dec 2011 at 8:56

[GoogleCodeExporter]

You are welcome :)

Original comment by flier...@gmail.com on 1 Dec 2011 at 9:01

• Changed state: Verified