-
Notifications
You must be signed in to change notification settings - Fork 5
Attempt to make this work with nwjs 0.13.3 #6
Comments
Thanks for the report. It's been a while since I've updated this so I'll give it a shot as soon as I can to see if I can get it working with latest version of NW.js. |
I also need nwjs 0.13 5.0.0 edge, and I want to finish the job myself.
|
@longang520, the error you report is normally because the compiled native module is not able to be loaded into the version of NWjs that you have. Could be mismatched 32bit / 64bit or something else. I have not tried with latest NWjs yet so don't know what changes will need to take place in order to make it work (yet). |
@longang520 I was able to reproduce your error with nwjs-sdk-v0.13.3-win-ia32, to get rid of it I rebuild edge as x86 (which, i assume, by default is being downloaded as x64 version):
I think that you could do the same, but only:
nw-gyp configure --target=v0.13.3 --msvs_version=2013 --arch=x64
P.S.:and after that i'm back to my main problem with "property 'getFileName' "). |
@VRastrigin |
@VRastrigin |
@VRastrigin |
@longang520 I'd also checked if there are installed (they, who use nw.js usually recommend it), and if not, install:
|
@VRastrigin |
@longang520 no problem, so it's very interesting to me if your NW.JS+edge.js compiled example shows you your c# string? |
I'm going to play with this today. I did notice on the NWjs documentation this: http://docs.nwjs.io/en/latest/For%20Users/Advanced/Use%20Native%20Node%20Modules/
@longang520, I see your comment about that you got it to compile and work. Are you saying that the example is totally working for you in the latest version of NWjs? |
Yes, the latest version of 0.13.3 nwjs, do you need an example? |
I'd love to know the steps you used to build the module. The documentation link I just pasted above is a bit misleading and it says that nw-gyp is no longer needed for native modules but they go on to mention how to build modules still using nw-gyp. I am currently trying to get it to work without using nw-gyp but have not succeeded yet. |
Your step is not successful, send out your problem, I can help solve |
@longang520 i confirm, mine works too: nw.js 0.13.3 x86 + edge 5.0.0 x86 |
Trying this now. Thanks! |
Yeah it all works. Thanks for mentioning your steps. I substituted 32bit for 64 in my case and still works fine. I appreciate your help. I'll update the README. |
README has been updated via 3afb188 THANK YOU FOR YOUR HELP!!! |
My configuration:
...\TestApp_new_nwjs-sdk-v0.13.3-win-x64\TestApp\node_modules\edge\lib\edge.js:79 Uncaught TypeError: Cannot read property 'getFileName' of undefinedexports.func @
...\TestApp_new_nwjs-sdk-v0.13.3-win-x64\TestApp\node_modules\edge\lib\edge.js:79
79th string of edge.js looks like:
" options = { source: options, jsFileName: stack[1].getFileName(), jsLineNumber: stack[1].getLineNumber() }; "
What i change from instruction:
"nw-gyp configure --target=v0.13.3 --msvs_version=2013"
i create new dir ...\TestApp_new_nwjs-sdk-v0.13.3-win-x64\TestApp\node_modules\edge\lib\native\win32\x64\4.4.2
(because i have node.js 4.4.2)
and copy these files there
...\TestApp_new_nwjs-sdk-v0.13.3-win-x64\TestApp\node_modules\edge\lib\edge.js:
[ /^0.8./, '0.8.22' ],
[ /^0.10./, '0.10.0' ],
[ /^0.12./, '0.12.0' ],
[ /^4./, '4.1.1' ],
[ /^4.4.2/, '4.4.2' ], // i add this row
[ /^5./, '5.1.0' ],
Any suggestions?
The text was updated successfully, but these errors were encountered: