search

josephg / node-foundationdb

Modern Node.js FoundationDB bindings
Other
116 stars 17 forks source link

Won't install on ubuntu for windows #25

Closed BEllis closed 5 years ago

BEllis commented 5 years ago

I get the following make error, I'm assuming I need to install a library or apk package but I can't find anything suitable. I installed foundationDB for windows, but I guess the libraries are .dlls for the windows version?

make: Entering directory '/mnt/c/dev/.../node_modules/foundationdb/build' CXX(target) Release/obj.target/fdblib/src/FdbV8Wrapper.o In file included from ../src/Database.h:28:0, from ../src/FdbV8Wrapper.cpp:31: ../src/Transaction.h:29:32: fatal error: foundationdb/fdb_c.h: No such file or directory compilation terminated. fdblib.target.mk:109: recipe for target 'Release/obj.target/fdblib/src/FdbV8Wrapper.o' failed make: *** [Release/obj.target/fdblib/src/FdbV8Wrapper.o] Error 1 make: Leaving directory '/mnt/c/dev/.../node_modules/foundationdb/build'

josephg commented 5 years ago

Hm. You’ll need to install the Foundationdb client library dpkg inside Linux for windows. But as for the error, I’m surprised - it should just be using the normal Linux prebuilds. It shouldn’t need to compile at all.

josephg commented 5 years ago

I just tried this on ubuntu 18.04 on windows. I ran into a similar error, but it went away after I installed the foundationdb-clients dpkg file from the foundationdb download page.

It looks like this package is trying to load the linux foundationdb client library. When that fails, it assumes it hasn't been compiled for your platform and then it tries to re-compile everything. And the recompilation fails because you're missing the headers - since again, you don't have the client installed.

So running this made it work just fine for me:

$ wget 'https://www.foundationdb.org/downloads/6.0.18/ubuntu/installers/foundationdb-clients_6.0.18-1_amd64.deb'
$ sudo dpkg -i foundationdb-clients_6.0.18-1_amd64.deb
$ npm i foundationdb

Let me know if you still have problems. Otherwise close this issue

josephg commented 5 years ago

Closing due to inactivity by author. Re-open if needed.