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

publish napi-maybe-compressed-blob-linux-arm64-gnu package to npm to support arm linux #647

Closed
dzmitry-lahoda opened this issue Dec 12, 2022 · 8 comments

Comments

@dzmitry-lahoda
Copy link
Contributor

> zombienet@0.1.0 zombie
> node ./packages/cli/dist/cli.js spawn /home/dom/rococo-local-dali-dev.toml

/nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/node_modules/napi-maybe-compressed-blob/index.js:208
    throw loadError
    ^

Error: Cannot find module 'napi-maybe-compressed-blob-linux-arm64-gnu'
Require stack:
- /nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/node_modules/napi-maybe-compressed-blob/index.js
- /nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/packages/orchestrator/dist/jsapi-helpers/chain-upgrade.js
- /nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/packages/orchestrator/dist/jsapi-helpers/index.js
- /nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/packages/orchestrator/dist/index.js
- /nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/packages/cli/dist/cli.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:985:15)
    at Module._load (node:internal/modules/cjs/loader:833:27)
    at Module.require (node:internal/modules/cjs/loader:1051:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/node_modules/napi-maybe-compressed-blob/index.js:177:31)
    at Module._compile (node:internal/modules/cjs/loader:1149:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1203:10)
    at Module.load (node:internal/modules/cjs/loader:1027:32)
    at Module._load (node:internal/modules/cjs/loader:868:12)
    at Module.require (node:internal/modules/cjs/loader:1051:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/node_modules/napi-maybe-compressed-blob/index.js',
    '/nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/packages/orchestrator/dist/jsapi-helpers/chain-upgrade.js',
    '/nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/packages/orchestrator/dist/jsapi-helpers/index.js',
    '/nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/packages/orchestrator/dist/index.js',
    '/nix/store/r628b9g10l3q2rsmb8x8ydqlgj5lwyb6-zombienet/packages/cli/dist/cli.js'
  ]
}
@dzmitry-lahoda
Copy link
Contributor Author

as far as I see darwin arm is here

@dzmitry-lahoda
Copy link
Contributor Author

I see it was published out of local folder, not from github https://www.npmjs.com/package/napi-maybe-compressed-blob?activeTab=explore

@dzmitry-lahoda
Copy link
Contributor Author

@pepoviola may you help or should we fork and republish our fork?

@pepoviola
Copy link
Collaborator

Hi @dzmitry-lahoda, yes I will publish today for arm.
Thanks!

@dzmitry-lahoda
Copy link
Contributor Author

dzmitry-lahoda commented Dec 13, 2022

 isMusl = readFileSync('/usr/bin/ldd', 'utf8').includes('musl')

not for this item, i will overcome it via link. but not all oses has /usr/bin/ldd. example nixos has it /bin/ldd. which is in PATH, so can be found just via ldd

just comment so be aware of handing stuff like that overall

@pepoviola
Copy link
Collaborator

Hi @dzmitry-lahoda, yes. We already hit that issue and @gilescope open an issue in napi-rs (napi-rs/napi-rs#1335).

Thanks!!

@pepoviola
Copy link
Collaborator

pepoviola commented Dec 13, 2022

@pepoviola
Copy link
Collaborator

 isMusl = readFileSync('/usr/bin/ldd', 'utf8').includes('musl')

not for this item, i will overcome it via link. but not all oses has /usr/bin/ldd. example nixos has it /bin/ldd. which is in PATH, so can be found just via ldd

just comment so be aware of handing stuff like that overall

upstream pr as reference napi-rs/napi-rs#1391

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

2 participants