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

Upgrade to Ocaml 4.10 and Core 0.14 (remove OCaml 4.07) #878

Merged
merged 3 commits into from
Aug 3, 2020

Conversation

anton-trunov
Copy link
Contributor

@anton-trunov anton-trunov commented Aug 3, 2020

Disable OCaml 4.07 because the new version of Core is not compatible
with it, hence we are free to use the new features provided by
4.08 (no new feature is used as of now).

Anton Trunov added 3 commits August 3, 2020 11:51
Fix compilation errors and warnings mostly resulting
from the Core upgrade

Disable OCaml 4.07 because the new version of Core is not compatible
with it, hence we are free to use the new features provided by
4.08 (no new feature is used as of now)
Recently Core_kernel switched to
Int.Replace_polymorphic_compare switched on by default
(open Core_kernel already has this effect)

The command I used to deal with the most of changes (GNU sed version, gsed):
find ./src ./tests -type f -iname '*.ml' -print0 | xargs -0 gsed  -i '/^open! Int.Replace_polymorphic_compare/d'
Copy link
Contributor

@jjcnn jjcnn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

I've run make and make test using OCaml v4.08.1, and everything seems to check out.

@anton-trunov anton-trunov merged commit 86f796a into master Aug 3, 2020
@anton-trunov anton-trunov deleted the ocaml-4.10 branch August 3, 2020 12:02
vaivaswatha added a commit to Zilliqa/scilla-compiler that referenced this pull request Aug 3, 2020
vaivaswatha added a commit to Zilliqa/scilla-compiler that referenced this pull request Aug 3, 2020
* Upgrade to Ocaml 4.10 and Core 0.14 (remove OCaml 4.07)

See Zilliqa/scilla#878

* Fix typo in compiler version 4.10.0
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.

None yet

2 participants