search

paldepind / synceddb

Makes it easy to write offline-first applications with realtime syncing and server side persistence.
MIT License
404 stars 34 forks source link

ws 0.7.0 dependency doesn't work with node 5.1? #8

Closed arlair closed 8 years ago

arlair commented 8 years ago

Just letting you know I had to change this because bufferutil 1.1.x does not seem to compile in Node 5.1 (and sounds like maybe 4.* from other comments I saw, but not definite about that)

I used 0.8.0 and it compiled.

I think it was in two of the synceddb packages.

piglovesyou commented 8 years ago

I pass all tests with ws 0.7.0 and node v5.8.0. Weird.

But we should use ws v1.0.0 or later that seems more stable.

piglovesyou commented 8 years ago

I reproduced. Something wrong with one published on npm?

All log on gist

piglovesyou commented 8 years ago

CI needs another module to be set

piglovesyou commented 8 years ago

@arlair Thank you for the report.

@paldepind, We need to publish synceddb-server with the newer dependency.