Skip to content

[v1.x] Backport of "#100: Update for 5.4"#104

Merged
ejgallego merged 2 commits into
ocaml-ppx:v1.xfrom
ejgallego:ocaml_54_1x
Oct 13, 2025
Merged

[v1.x] Backport of "#100: Update for 5.4"#104
ejgallego merged 2 commits into
ocaml-ppx:v1.xfrom
ejgallego:ocaml_54_1x

Conversation

@ejgallego
Copy link
Copy Markdown
Collaborator

No description provided.

@ejgallego
Copy link
Copy Markdown
Collaborator Author

4.05 - 4.07 fail due to missing Pprintast.longident function cc @gasche @Octachron @kit-ty-kate

Should we bump the opam file, or do you folks thing that we should make this work on 4.05 ?

@kit-ty-kate
Copy link
Copy Markdown
Contributor

The new consensual lower bound in the community is OCaml 4.08 at the moment. opam-repository has stopped supporting OCaml < 4.08 earlier this year, so i would suggest simply bumping the constraint

@ejgallego
Copy link
Copy Markdown
Collaborator Author

Thank a lot for the feedback, makes indeed sense. I will do this, cheers.

@ejgallego ejgallego merged commit 2a28839 into ocaml-ppx:v1.x Oct 13, 2025
14 of 17 checks passed
@ejgallego ejgallego deleted the ocaml_54_1x branch October 13, 2025 17:09
ejgallego added a commit to ejgallego/opam-repository that referenced this pull request Oct 13, 2025
CHANGES:

  * Support for OCaml 5.4 (ocaml-ppx/ppx_import#100, @Octachron, backport ocaml-ppx/ppx_import#104, @egallego)
  * Support for OCaml 4.5, 4.6, and 4.7 has been removed (ocaml-ppx/ppx_import#104, @egallego)
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