Skip to content

support OCaml 5.5#510

Merged
Octachron merged 1 commit into
ocaml-community:masterfrom
anmonteiro:anmonteiro/ocaml-5.5-support
Mar 6, 2026
Merged

support OCaml 5.5#510
Octachron merged 1 commit into
ocaml-community:masterfrom
anmonteiro:anmonteiro/ocaml-5.5-support

Conversation

@anmonteiro

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
@anmonteiro
anmonteiro force-pushed the anmonteiro/ocaml-5.5-support branch from d9cfc4e to 629b067 Compare February 15, 2026 20:14

@Octachron Octachron left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is good to merge from my point of view.

Comment thread src/lib/uTop.ml
#if OCAML_VERSION >= (5, 5, 0)
let si = Str.module_ ~loc (Mb.mk ~loc (with_loc loc (Some "_")) (Mod.structure (item :: items))) in
let body = (Exp.struct_item ~loc si (Exp.construct unit None)) in
#else

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am wondering if we should have a helper letmodule function to make it clearer than the two branches are the same but for one use point this seems good enough.

@Octachron

Copy link
Copy Markdown
Member

I am proposing of merging now, and we will publish the new utop version after the first beta for 5.5.0 .

@kit-ty-kate

Copy link
Copy Markdown
Member

ping @Octachron

@Octachron
Octachron merged commit ce32aa4 into ocaml-community:master Mar 6, 2026
3 checks passed
Octachron added a commit to Octachron/opam-repository that referenced this pull request Mar 26, 2026
CHANGES:

* Add support for OCaml 5.5 (ocaml-community/utop#510 @anmonteiro)
* Add support for neocaml or other major modes in utop.el (ocaml-community/utop#511, @bbatsov)
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.

3 participants