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

install failure due to #71

Open
erkinalp opened this issue Aug 29, 2024 · 8 comments
Open

install failure due to #71

erkinalp opened this issue Aug 29, 2024 · 8 comments

Comments

@erkinalp
Copy link

npm WARN deprecated [email protected]: This package is no longer supported.
npm WARN deprecated [email protected]: This package is no longer supported.
npm WARN deprecated [email protected]: This package is no longer supported.
npm ERR! code 1
npm ERR! path /usr/local/lib/node_modules/humanifyjs/node_modules/isolated-vm
npm ERR! command failed
npm ERR! command sh -c prebuild-install || (node-gyp rebuild --release -j max && node-gyp clean)
@0xdevalias
Copy link
Contributor

See also:


I don't think those deprecated packages are actually the source of your install failure here though; I think it's more related to isolated-vm.

Are you able to share more details about what system you are running this on, what version of node, etc?

I know isolated-vm is used by @j4k0xb 's webcrack (which this project uses), and I know I have seen other isolated-vm issues there before, so it might also help to have a look on there too:

@erkinalp
Copy link
Author

Node 18.19
Ubuntu 24.04 (x86)
GNOME 46

@jehna
Copy link
Owner

jehna commented Aug 31, 2024

Yep, you'll need node.js >= 20, the prebuilt binaries are available for those versions only.

@jehna
Copy link
Owner

jehna commented Aug 31, 2024

I've found the nodesource distributions to be a straightforward way to install more recent node versions on debian

@jehna
Copy link
Owner

jehna commented Aug 31, 2024

There is a note about needing node.js 20 in the readme, but it seems to be easy to miss. I wonder which place would be better. Maybe a preinstall hook thst ensures the version?

@0xdevalias
Copy link
Contributor

0xdevalias commented Aug 31, 2024

I wonder which place would be better.

@jehna Does the package.json engines field enforce it?

Sounds like it may only work when installed as a dependency, so maybe not when installed directly?


you'll need node.js >= 20, the prebuilt binaries are available for those versions only

There is a note about needing node.js 20 in the readme

@jehna What specifically needs node 20, is it humanify itself, or one of its dependencies? If dependency, I wonder if it would make sense to try and get the upstream projects to add the engines field?


@jehna Could also maybe use preinstall script in package.json?

@neoOpus
Copy link

neoOpus commented Sep 4, 2024

There is a note about needing node.js 20 in the readme, but it seems to be easy to miss. I wonder which place would be better. Maybe a preinstall hook thst ensures the version?

So, you advise using this in WSL instead of directly on Windows NodeJS ?

@jehna
Copy link
Owner

jehna commented Sep 4, 2024

Unfortunately I have no idea how Windows works on development, the last time I used Windows as my main dev machine was over 10 years ago 😅

I'd guess both WSL and non-wsl should work fine, as long as you have recent enough version. You can use e.g. nvm to switch between versions on WSL

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