Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot build from source: Cohttp_async_io is unbounded (OS X 10.11.6) #75

Closed
erwanor opened this issue Sep 8, 2017 · 5 comments
Closed
Labels

Comments

@erwanor
Copy link

erwanor commented Sep 8, 2017

When trying to build from sources with tests enabled:

./configure --enable-examples
make clean && make

yields the following error:

Zoom: Error: Unbound module Cohttp_async_io

context:

File "examples/hello_async.ml", line 1, characters 5-13:
Warning 3: deprecated: module Core.Std
[since 2017-02] Use Core. The Std sub-module is no longer needed
File "examples/hello_async.ml", line 2, characters 5-14:
Warning 3: deprecated: module Async.Std
[since 2017-02] use Async. The Std sub-module is no longer needed
File "examples/hello_async.ml", line 5, characters 5-20:
Error: Unbound module Cohttp_async_io
Hint: Did you mean Cohttp_async?
Command exited with code 2.

even after I run opam install --deps-only webmachine:

$ opam install --deps-only webmachine

=-=- Synchronising pinned packages =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=  🐫
[webmachine] /Users/erwan/ocaml-webmachine/ already up-to-date

My uname -a :

Darwin bastiat.local 15.6.0 Darwin Kernel Version 15.6.0: Thu Jun 23 18:25:34 PDT 2016; root:xnu-3248.60.10~1/RELEASE_X86_64 x86_64

My ocaml --version:

 The OCaml toplevel, version 4.04.2

I obtain the same error when install from opam (via opam install webmachine).

@seliopou seliopou added the bug label Sep 8, 2017
@seliopou
Copy link
Member

seliopou commented Sep 8, 2017 via email

@rgrinberg
Copy link

I think Cohttp 0.99 changed this to Cohttp_async.Io (when we moved it to aliases). I did a rev dep search for possible breakage of this but i think the depopts made me miss this case..

@tmcgilchrist
Copy link
Contributor

tmcgilchrist commented Sep 8, 2017 via email

tmcgilchrist added a commit to tmcgilchrist/ocaml-webmachine that referenced this issue Sep 8, 2017
@seliopou
Copy link
Member

seliopou commented Sep 8, 2017

This should be fixed by the patch from @tmcgilchrist.

@seliopou seliopou closed this as completed Sep 8, 2017
@erwanor
Copy link
Author

erwanor commented Sep 8, 2017

@tmcgilchrist thanks for the swift action!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants