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

Add some clarifications to the README on build dependencies #12

Merged
merged 1 commit into from
Nov 18, 2014

Conversation

avsm
Copy link
Contributor

@avsm avsm commented Nov 18, 2014

  • Minimum OCaml compiler version is 4.01.0+, not 4.x
  • OPAM package may also be convenient for some users

You may not want all the detail in the latter, but I thought it
would be useful for those users stuck on ancient distros like
CentOS that ship with OCaml 3.12.1.

The related OPAM pull request: ocaml/opam-repository#3083

- Minimum OCaml compiler version is 4.01.0+, not 4.x
- OPAM package may also be convenient for some users

You may not want all the detail in the latter, but I thought it
would be useful for those users stuck on ancient distros like
CentOS that ship with OCaml 3.12.1.

The related OPAM pull request: ocaml/opam-repository#3083
@gabelevi
Copy link
Contributor

Great, thanks for writing this up! I'll merge this as soon as the package is visible to opam (I don't see the new flow package when I run opam update && opam info flow)

@gabelevi
Copy link
Contributor

Err make that opam info flowtype

@avsm
Copy link
Contributor Author

avsm commented Nov 18, 2014

Should be there now -- the central repo updates hourly.

On 18 Nov 2014, at 21:17, Gabe Levi [email protected] wrote:

Err make that opam info flowtype


Reply to this email directly or view it on GitHub.

@gabelevi
Copy link
Contributor

There it is! Awesome!

gabelevi added a commit that referenced this pull request Nov 18, 2014
Add some clarifications to the README on build dependencies
@gabelevi gabelevi merged commit dc111d1 into facebook:master Nov 18, 2014
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

Successfully merging this pull request may close these issues.

2 participants