npm ERR! gyp: Undefined variable android_ndk_path in binding.gyp while trying to load binding.gyp about gyp-next HOT 7 CLOSED

Error log?

from gyp-next.

2024-03-27T16_39_28_462Z-debug-0.log

It looks like node-gyp add some gypi contains this variable:

$ rg android_ndk_path /data/data/com.termux/files/home/.cache/node-gyp/21.6.2/include/node/common.gypi
241:            'cflags': [ '-fPIC', '-I<(android_ndk_path)/sources/android/cpufeatures' ],

from gyp-next.

It looks like the alpha software crashed. What happens when you remove that software from the build?

35 info run [email protected] install node-gyp rebuild
36 info run [email protected] install { code: 1, signal: null }
37 timing command:rebuild Completed in 1514ms
38 verbose stack Error: command failed
38 verbose stack at ChildProcess. (/data/data/com.termux/files/usr/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/lib/index.js:53:27)
38 verbose stack at ChildProcess.emit (node:events:519:28)
38 verbose stack at maybeClose (node:internal/child_process:1105:16)
38 verbose stack at ChildProcess._handle.onexit (node:internal/child_process:305:5)

from gyp-next.

Closing in favor of

• tonyfettes/coc-rime#7

from gyp-next.

After tonyfettes/coc-rime@c91900b, it can be build in Android/termux. So can we add a default android_ndk_path to avoid this bug?

PS: It should be added in node-gyp's js code?

from gyp-next.

The error messages on tonyfettes/coc-rime#7 say nothing about android_ndk_path. Let’s get that merged so that we have clear evidence of a android_ndk_path problem so that we can recruit other to fix that problem.

from gyp-next.

It's my fault. Let me explain the full steps:

1. Install https://github.com/termux/termux-app to an Android device

apt install nodejs librime json-c pkg-config
git clone --depth=1 https://github.com/tonyfettes/coc-rime
cd coc-rime
# before https://github.com/tonyfettes/coc-rime/commit/c91900b01ca84c1e43f53958dca30d3a59b23ab4
git checkout f307fa8c71a042663111705e14611444cacc2955
npm install
npm rebuild

You will set that error message.

from gyp-next.