-
Notifications
You must be signed in to change notification settings - Fork 176
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
cmake.js support #268
Comments
Well, Simply add an empty binding.gyp will work . |
@xVanTuring thank you for the slight work around. When I make your suggested change, it does build via cmake, but not at the correct Node Module Version on my end. Do you have any suggestions? I would also appreciate official support for cmake-js |
Are there any plans on adding this feature? |
This is on my TODO list, but it's pretty far down. We'd be happy to have a contribution to add support for this from the community, though! |
Any update on this? I have a native module that when built with cmake-js it works fine. Obviously that is just an issue on my end, but if electron-rebuild could use cmake-js it would solve my problem lol I tried the blank binding.gyp trick noted above, but electron-rebuild then says it can't parse the file :( |
Found the problem between node-gyp and cmake-js Adding them into my cmake-js then everything works and I can run my native module in electron compiling my module outside of electron-rebuild and then just bringing it in manually. Figured I would close the loop on this; other people might come here with issues. |
I am trying to rebuild aws-crt so it will work with electron. @xVanTuring said adding an empty @smalls12 what exactly did you do to make your native module work? |
An empty
|
Is cmake-js supported now? |
Is there a final workaround for how to use |
CMake.js is quite awesome and easy to use. Even used in Microsoft/napajs!
It will be greate to add support for cmake.js.
Currently electron-rebuild can only recognize the
binding.gyp
file, I think it should be quite easy to add support for cmake.js.The text was updated successfully, but these errors were encountered: