nodejs / llnode

An lldb plugin for Node.js and V8, which enables inspection of JavaScript states for insights into Node.js processes and their core dumps.
Other
1.15k stars 100 forks source link

Could not find any usable lldb executable #289

Open lcw0622 opened 5 years ago

lcw0622 commented 5 years ago

An error occured while executing npm install -g llnode, and I had already installed lldb. There is output:

[root@63675c114bd6 llnode]# npm install -g llnode

> llnode@2.2.0 preinstall /usr/local/lib/node_modules/llnode
> node scripts/configure.js

Build dir is: /usr/local/lib/node_modules/llnode
Build dir is: /root
Looking for lldb executable...
Could not find any usable lldb executable

I find the reason : preinstall script scripts/lldb.js use command which to locate command lldb's path in function tryExecutables, but in Centos core version, they only have whereis. So I'm doubt is it necessary to add whereis support?

OS: CentOS Linux release 7.2.1511 (Core) in Docker lldb: lldb version 3.4.2 ( revision )

lcw0622 commented 5 years ago

And whereis execute in CentOS results:

[root@63675c114bd6 ~]# whereis lldb
lldb: /usr/bin/lldb /usr/include/lldb /usr/share/man/man1/lldb.1.gz
joyeecheung commented 5 years ago

Add fallback with whereis for CentOS sounds good. Do you mind sending in a PR?

tusharmctrl commented 5 months ago

Hey, been a long time - were you able to fix this one?