You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
During today's call we discussed the behavior of CMake.js when the underlying CMake executable is not installed. Here's what I found:
spare:~ jim$ npm install -g cmake-js
/Users/jim/.nvm/versions/node/v10.16.3/bin/cmake-js -> /Users/jim/.nvm/versions/node/v10.16.3/lib/node_modules/cmake-js/bin/cmake-js
+ cmake-js@6.0.0
added 140 packages from 108 contributors in 8.486s
spare:~ jim$ cmake
-bash: cmake: command not found
spare:~ jim$ cmake-js
[ '/Users/jim/.nvm/versions/node/v10.16.3/bin/node',
'/Users/jim/.nvm/versions/node/v10.16.3/bin/cmake-js' ]
info TOOL Using Unix Makefiles generator.
ERR! OMG CMake executable is not found. Please use your system's package manager to install it, or you can get installers from there: http://cmake.org.
spare:~ jim$
CMake.js displays a message instructing the user to install the CMake executable if it is not found.
Activity
mhdawson commentedon Sep 16, 2019
Nicola, we should look at existing solutions like
cmake-jsand see if they need any updates to support node-addon-api.Jim, the work he did for pre-build was for cmake-js.
Nicola, maybe we can move some of what Jim did for pre-build to cmake-js so that all dependencies can leverage that.
Jim, cmake-js is pretty generic, did not have to do much, only document what you needed there.
Nicola then it might just be a matter of documentation.
Jim will:
gengjiawen commentedon Sep 19, 2019
node-gypalready support generate cmake config. I have used it in Clion, works pretty well (code autocomplete and debug).jschlight commentedon Sep 27, 2019
I've added a comment here about building N-API addons using CMake.js: nodejs/node-addon-api#445 (comment)
jschlight commentedon Sep 30, 2019
During today's call we discussed the behavior of CMake.js when the underlying CMake executable is not installed. Here's what I found:
CMake.js displays a message instructing the user to install the CMake executable if it is not found.
mhdawson commentedon Oct 1, 2019
@jschlight thanks for checking that out. Great to see that it shows a good message.
jschlight commentedon Oct 24, 2019
The PR merge should have closed this issue? But maybe the automatic close does not work between repos?