Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build fails if your node binary isn't called nodejs #1

Closed
jonaslund opened this issue Mar 28, 2019 · 6 comments
Closed

Build fails if your node binary isn't called nodejs #1

jonaslund opened this issue Mar 28, 2019 · 6 comments

Comments

@jonaslund
Copy link

Easily fixed by changing this line in binding.gyp
"include_dirs": [ "./external/matrix/include", "./include/", "<!(nodejs -e \"require('nan')\")" ]

Should be fixed

@keptan
Copy link
Collaborator

keptan commented Apr 11, 2019

thanks ill look into this

@redandinvisible
Copy link

I am having the same problem. Is there a workaround by any chance?

@redandinvisible
Copy link

In case it helps others:

I cloned this repo and made it a local dependency. Then I changed the line in the local binding.gyp so the node binary was just called node. Tried npm install again which gave a new error make: *** No rule to make target `Release/obj.target/rpi-rgb-led-matrix/external/matrix/lib/transformer.o', needed by `Release/rpi-rgb-led-matrix.a'..

Turns out this was caused by some clever cross-github linking in the external directory to this "rpi-rgb-led-matrix" github repo. So I cloned the contents of that repo into the external directory. Removed the uneeded .git directories. And it now builds.

@samturner3
Copy link

+1
Screen Shot 2019-05-28 at 3 17 38 pm

@keptan
Copy link
Collaborator

keptan commented May 28, 2019

@redandinvisible the second half of your issue is just a matter of checking out submodules, I should make that clearer in the readme probably
when you pull from npm this isn't an issue

still haven't changed the binding.gyp sorry

@keptan
Copy link
Collaborator

keptan commented May 28, 2019

e4bae0b

finally dealing with this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants