ironm73 / pyv8

Automatically exported from code.google.com/p/pyv8
0 stars 0 forks source link

Compilation error due to V8 changes #104

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

While compiling PyV8 r394 I got the following error

In file included from src/Engine.cpp:36:
src/AST.h: In function ‘boost::python::api::object 
to_python(v8::internal::Handle<v8::internal::String>)’:
src/AST.h:41: error: ‘class v8::internal::String’ has no member named 
‘ToAsciiVector’
src/AST.h: In function ‘const std::string 
to_string(v8::internal::Handle<v8::internal::String>)’:
src/AST.h:50: error: ‘class v8::internal::String’ has no member named 
‘ToAsciiVector’
error: command 'x86_64-pc-linux-gnu-g++' failed with exit status 1

The problem is due to a change in V8 committed in r8999

http://code.google.com/p/v8/source/detail?r=8999

Original issue reported on code.google.com by angelo.d...@gmail.com on 25 Aug 2011 at 8:21

GoogleCodeExporter commented 8 years ago
Realized just now I opened a duplicate issue notification. Sorry.

Original comment by angelo.d...@gmail.com on 25 Aug 2011 at 8:23

GoogleCodeExporter commented 8 years ago
I am definitely seeing this issue as well.

Original comment by treyl...@jess3.com on 25 Aug 2011 at 4:06

GoogleCodeExporter commented 8 years ago
While waiting for a fix, you can build pyv8 trunk if you revert to the 3.5.7 
tag of v8:

    $ svn co http://v8.googlecode.com/svn/tags/3.5.7 v8-3.5.7
    $ export V8_HOME=$PWD/v8-3.5.7
    $ python setup.py build

Everything after this tag will contain the incompatible change (8999), as the 
next tag is at 9009: http://code.google.com/p/v8/source/detail?r=9009

The tests all pass except testDate due to issue #47. 
http://code.google.com/p/pyv8/issues/detail?id=47

Original comment by david.schoonover on 26 Aug 2011 at 2:26

GoogleCodeExporter commented 8 years ago
Please verify it with SVN trunk code after r395

Original comment by flier...@gmail.com on 26 Aug 2011 at 5:37

GoogleCodeExporter commented 8 years ago
Confirmed, and all tests pass (excepting issue #47).

Original comment by david.schoonover on 26 Aug 2011 at 11:16

GoogleCodeExporter commented 8 years ago

Original comment by flier...@gmail.com on 27 Aug 2011 at 2:07