-
-
Notifications
You must be signed in to change notification settings - Fork 7
Building issue on Windows platform #5
Comments
The problem is that ELEMENT_TYPE is already defined in winioctl.h. |
There is another problem. Array of variable length are not supported. binding.cc:124 could be replaced with: current: char cookie[cookieSize]; |
I just get syntax errors (a lot of them)
I know I CAN build packages, because mdns builds fine. (I tried installing nodetunes) Node version is 0.12.9 |
I was able to compile it using VS 2015 Community Edition ... but Airsonos wasn't accepted by iTunes as AirPlay device. I only succeeded to compile it on Ubuntu with microadam's nodetunes patch and was able to use it with iTunes. https://github.com/m-a-v/airsonos which depends on: I don't remember the Node version i used. |
microadams version seems to be merged upstream. Added pull-request with a fix for the errors above #6 |
I didn't remember anymore that microadams was merged upstream but I noticed that I didn't use the microadam version mentioned above. I tried it to recompile with VS 2015 and Node 0.12.9 (x64) with success. But only if I append "--no-bin-links". |
I get the following error when building without --no-bin-links. Anyone can help me? 39186 verbose unlock done using C:\Users\Username\AppData\Roaming\npm-cache_locks\airsonos-f94f1b5423456247.lock for C:\Users\Username\AppData\Roaming\npm\node_modules\airsonos |
the same issues |
Sorry, this is unmaintained. I will archive the repository now. |
The lib apparently fails to npm install on a Windows platform with bitutilities / binding definitions ?
\node_modules\libalac\ALAC\codec\ALACBitUtilities.h(68): error C2371: 'ELEMENT_TYPE'
..\binding.cc(124): error C2131 [\node_modules\libalac\build\binding.vcxproj]
The text was updated successfully, but these errors were encountered: