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 More OCaml to Books page #554

Closed
amirmc opened this issue Oct 22, 2014 · 6 comments
Closed

Add More OCaml to Books page #554

amirmc opened this issue Oct 22, 2014 · 6 comments

Comments

@amirmc
Copy link
Member

amirmc commented Oct 22, 2014

http://ocaml-book.com/more-ocaml-algorithms-methods-diversions/

@agarwal
Copy link
Member

agarwal commented Oct 22, 2014

Related to #524.

@dbuenzli
Copy link
Contributor

I'm currently going through the book and (somehow as expected) it's an excellent one. It's a shame it's still not mentioned on the website. @johnwhitington I don't think anyone will blame you if you take action to integrate it yourself.

@johnwhitington
Copy link
Contributor

I've had a go at getting ocaml.org to build on my machine so I can begin to make contributions, but no luck yet (stuck on getting ocamlnet to build for now).

Perhaps the root of the problem is that something in "make deps" seems to rely on OCaml being exactly version 4.01.0, so everything I'm doing may be out of date. Can that requirement really still be needed?

=-=- Synchronising pinned packages =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=  🐫 
[ocamlorg] /Users/john/repos/ocaml.org/ already up-to-date
[ERROR] ocamlorg is not available because your system doesn't comply with
        ocaml-version = "4.01.0".

@dbuenzli
Copy link
Contributor

@johnwhitington You can keep this simple by simply using the pencil on the top right corner visible on any page of the website. This will lead you to github where you can simply edit the .md file by clicking again the pencil. I guess in this case you will maybe need to make a separate commit for adding the image file to the img directory.

@johnwhitington
Copy link
Contributor

See Pull Request #657

@dsheets
Copy link
Member

dsheets commented May 26, 2015

Fixed by #657.

@dsheets dsheets closed this as completed May 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants