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

Cannot npm i node@8 on an armv7l machine #57

Open
losingkeys opened this issue Jan 7, 2019 · 12 comments
Open

Cannot npm i node@8 on an armv7l machine #57

losingkeys opened this issue Jan 7, 2019 · 12 comments

Comments

@losingkeys
Copy link

I've made a demo repo for this issue.

I think this may be related to #53, but I'm not sure. Please let me know if you need more information than the stuff in my demo repo. Thanks!

@aredridel
Copy link
Owner

I'm not on armv7l, but what's the error you get?

@losingkeys
Copy link
Author

Error: Cannot find module 'node-linux-arm/package.json' (logs here).

There's a longer explanation in the demo project's README. Let me know if you can't access it or need more info :).

@aredridel
Copy link
Owner

What's the value of process.arch on your platform?

@losingkeys
Copy link
Author

'arm'

@aredridel
Copy link
Owner

Well fudge. Is there one that supplies the 'v7l' ?

@losingkeys
Copy link
Author

Hmm... arch responds with armv7l.. but I'm having trouble getting even 7 from node... Here are some threads tracking the same issue:

  1. Host arch ARMv7 not correctly detected electron/packager#523 (comment)
  2. process.arch not matching with systems arch output for ARM nodejs/node#9491
  • Note: I get the same output as this commenter, i.e. uname -m -> armv7l and node -p 'process.config.variables.arm_version' -> 6

Could you run the external commands uname -m or arch via node? I know that's nasty.. but node -p 'process' | grep arm and similar exploratory commands didn't turn up anything useful for me :(

@aredridel
Copy link
Owner

What about os.arch() an os.platform() ?

@aredridel
Copy link
Owner

Shelling out to uname isn't the worst thing. It's done only at install time, and that script is where the mess Should live.

@losingkeys
Copy link
Author

  • os.arch() -> 'arm'
  • os.process() -> 'linux'

Good point about it being in an install script. Seems like an acceptable solution. I didn't find much in the linked threads, though I definitely didn't read all of them.

@aredridel
Copy link
Owner

Geez, node :(

@losingkeys
Copy link
Author

Would you be okay with me taking a crack at using the external program arch in the build setup? I'm not exactly sure how to do this, but will take a look if you like that solution.

@aredridel
Copy link
Owner

Yeah, either arch or uname, go for it!

losingkeys pushed a commit to losingkeys/node-bin-gen that referenced this issue Jan 14, 2019
See aredridel#57 for details and related threads
losingkeys pushed a commit to losingkeys/node-bin-setup that referenced this issue Jan 21, 2019
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

Successfully merging a pull request may close this issue.

2 participants