node-chakracore icon indicating copy to clipboard operation
node-chakracore copied to clipboard

node.gyp: hard coded include v8.h

Open jianchun opened this issue 8 years ago • 4 comments

Some recent changes caused hard coded v8.h:

https://github.com/nodejs/node-chakracore/blob/chakracore-master/node.gyp#L209

        'deps/v8/include/v8.h',
        'deps/v8/include/v8-debug.h',

jianchun avatar Jul 05 '16 20:07 jianchun

Please compare with master node.gyp. Some other parts may need sync too. e.g. official v8 deps is now authored as

        [ 'node_use_bundled_v8=="true"', {
          'dependencies': [
            'deps/v8/tools/gyp/v8.gyp:v8',
            'deps/v8/tools/gyp/v8.gyp:v8_libplatform'
          ],
        }],

jianchun avatar Jul 05 '16 20:07 jianchun

Some recent changes caused hard coded v8.h:

Seems that it was hard coded from day 1. I will look into it to see what can be done.

kunalspathak avatar Jul 05 '16 21:07 kunalspathak

I seem to recall moved them under node_engine=="v8" at some point, maybe change lost long ago.

jianchun avatar Jul 05 '16 21:07 jianchun

@kunalspathak Can you take a look?

kfarnung avatar Jun 14 '17 21:06 kfarnung