vbmithr / ocaml-websocket

Websocket library for OCaml
ISC License
163 stars 44 forks source link

Compilation error #2

Closed IgorLiner closed 11 years ago

IgorLiner commented 11 years ago

Any idea if lwt API was changed? I can't compile ocaml-websocket:

Any help would be appreciated.

vbmithr commented 11 years ago

My bad. I was using a special customized version of lwt (just a minor change) that never went upstream. I’m fixing that ASAP. Since people are interested I’m probably going to release this lib on OPAM!

IgorLiner commented 11 years ago

Thanks. Can you point me to the change? Can i apply this patch to lwt manually?

vbmithr commented 11 years ago

On 12/03/2013 18:16, IgorLiner wrote:

Thanks. Can you point me to the change? Can i apply this patch to lwt manually?

Yes. Either you use the branch "fix_compilation" (that fixes compilation, but that does not work correctly because I was using patched lwt to be ale to

let setup_socket fd = Lwt_unix.setsockopt fd Lwt_unix.TCP_NODELAY true

On the socket that is going to be used for the connection (websocket needs TCP_NODELAY).

Or you use my @vbmithr/lwt branch

(you can install it with opam by doing "opam pin git://")

I have to fix this anyway, I’ll do it soon. The correct thing to do is to not use Lwt_io.{open_connection,establish_server} but reimplementing them by a version that do the TCP_NODELAY.

Cheers

IgorLiner commented 11 years ago

Thanks, i'm currently using @vbmithr/lwt. Works great.