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

lwt_react.ml field bind is required but not provided #77

Closed
scottt opened this issue Aug 17, 2014 · 4 comments
Closed

lwt_react.ml field bind is required but not provided #77

scottt opened this issue Aug 17, 2014 · 4 comments

Comments

@scottt
Copy link

scottt commented Aug 17, 2014

Building lwt-2.4.5 fails when "--enable-react" is passed to configure

  1. git checkout 2.4.5
  2. ./configure --enable-react && make
    <...>
  3. /usr/bin/ocamlfind ocamlc -c -g -I src/core -package react -I src/react -I src/core -o src/react/lwt_react.cmo src/react/lwt_react.ml
    File "src/react/lwt_react.ml", line 1:
    Error: The implementation src/react/lwt_react.ml
    does not match the interface src/react/lwt_react.cmi:
    ...
    In module S:
    The field `bind' is required but not provided
    Command exited with code 2.

I believe this is caused by commit bec7372

@fccm
Copy link

fccm commented Oct 11, 2014

Try to update your react installation for a more recent version, it should solve this error.

@scottt
Copy link
Author

scottt commented Oct 11, 2014

@fccm, could you maintain a "known to work" react version in the lwt sources?

@fccm
Copy link

fccm commented Oct 13, 2014

@aantron
Copy link
Collaborator

aantron commented Jun 29, 2016

Indeed, this is solved by using React >= 1.0.0.

Lwt has since included an opam file in its distribution, so it should be easier to find the constraints.

@aantron aantron closed this as completed Jun 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants