chore: bump OCaml version 5.3 -> 5.4

Signed-off-by: Ali Caglayan <[email protected]>

Author: Alizter

.github/workflows/workflow.yml

@@ -168,22 +168,22 @@ jobs:
             os: ubuntu-latest
           # OCaml 5:
           ## ubuntu (x86)
-          - ocaml-compiler: 5.3.x
+          - ocaml-compiler: 5.4.x
             os: ubuntu-latest
             run_tests: true
           ## macos (Apple Silicon)
-          - ocaml-compiler: 5.3.x
+          - ocaml-compiler: 5.4.x
             os: macos-latest
             run_tests: true
           ## macos (x86)
           - ocaml-compiler: 5.4.x
             os: macos-15-intel
           ## MSVC
-          - ocaml-compiler: ocaml-compiler.5.3.0,system-msvc
+          - ocaml-compiler: ocaml-compiler.5.4.0,system-msvc
             os: windows-latest
             run_tests: true
           ## mingw
-          - ocaml-compiler: ocaml-base-compiler.5.3.0,system-mingw
+          - ocaml-compiler: ocaml-base-compiler.5.4.0,system-mingw
             os: windows-latest
             run_tests: true
           # OCaml 4:

Makefile

@@ -12,7 +12,7 @@ DEV_DEPS := \
 core_bench \
 patdiff
 
-TEST_OCAMLVERSION := 5.3.0
+TEST_OCAMLVERSION := 5.4.0
 # When updating this version, don't forget to also bump the number in the docs.
 
 -include Makefile.dev
@@ -118,9 +118,6 @@ test-melange: $(BIN)
 test-all: $(BIN)
 	$(BIN) build @runtest @runtest-js @runtest-coq @runtest-rocq @runtest-melange
 
-test-all-sans-melange: $(BIN)
-	$(BIN) build @runtest @runtest-js @runtest-coq @runtest-rocq
-
 test-ox: $(BIN)
 	$(BIN) runtest test/blackbox-tests/test-cases/oxcaml
 

doc/hacking.rst

@@ -54,10 +54,10 @@ Here are the most common commands you'll be running:
    $ ./dune.exe build @foo
 
 
-Note that tests are currently written for version 5.3.0 of the OCaml compiler.
+Note that tests are currently written for version 5.4.0 of the OCaml compiler.
 Some tests depend on the specific wording of compilation errors which can change
 between compiler versions, so to reliably run the tests make sure that
-``ocaml.5.3.0`` is installed. The ``TEST_OCAMLVERSION`` in the ``Makefile`` at
+``ocaml.5.4.0`` is installed. The ``TEST_OCAMLVERSION`` in the ``Makefile`` at
 the root of the Dune repo contains the current compiler version for which tests
 are written.
 

doc/tutorials/developing-with-dune/introduction.md

@@ -44,7 +44,7 @@ Let's create a local switch: `cd` to this directory and run the following comman
 This can take a few minutes.
 
 ```sh
-opam switch create ./ 5.3.0
+opam switch create ./ 5.4.0
 ```
 
 This command has created a directory named `_opam` in the current directory.

dune.opam

@@ -59,7 +59,7 @@ depends: [
   "ppx_expect" { with-dev-setup & >= "v0.17" & os != "win32" }
   "spawn" { with-dev-setup }
   "ppx_inline_test" { with-dev-setup & os != "win32" }
-  "ppxlib" { with-dev-setup & >= "0.35.0" & os != "win32" }
+  "ppxlib" { with-dev-setup & >= "0.37.0" & os != "win32" }
   "ctypes" { with-dev-setup & os != "win32" }
   "utop" { with-dev-setup & >= "2.6.0" & os != "win32" }
   "melange" { with-dev-setup & >= "5.1.0-51" & os != "win32" }

dune.opam.template

@@ -20,7 +20,7 @@ depends: [
   "ppx_expect" { with-dev-setup & >= "v0.17" & os != "win32" }
   "spawn" { with-dev-setup }
   "ppx_inline_test" { with-dev-setup & os != "win32" }
-  "ppxlib" { with-dev-setup & >= "0.35.0" & os != "win32" }
+  "ppxlib" { with-dev-setup & >= "0.37.0" & os != "win32" }
   "ctypes" { with-dev-setup & os != "win32" }
   "utop" { with-dev-setup & >= "2.6.0" & os != "win32" }
   "melange" { with-dev-setup & >= "5.1.0-51" & os != "win32" }

flake.lock

@@ -1,9 +1,10 @@
 {
   inputs = {
     nixpkgs.url = "github:nixos/nixpkgs/nixpkgs-unstable";
+    nixpkgs-old.url = "github:nixos/nixpkgs/7f50d4b33363d3948543f6a02b90a2c66852a453";
     flake-utils.url = "github:numtide/flake-utils";
     melange = {
-      url = "github:melange-re/melange/refs/tags/5.1.0-53";
+      url = "github:melange-re/melange/refs/tags/6.0.0-54";
       inputs.nixpkgs.follows = "nixpkgs";
     };
     ocaml-overlays = {
@@ -24,6 +25,7 @@
       self,
       flake-utils,
       nixpkgs,
+      nixpkgs-old,
       melange,
       ocaml-overlays,
       oxcaml,
@@ -35,7 +37,7 @@
         pkgs = nixpkgs.legacyPackages.${system}.appendOverlays [
           ocaml-overlays.overlays.default
           (self: super: {
-            ocamlPackages = super.ocaml-ng.ocamlPackages_5_3.overrideScope (
+            ocamlPackages = super.ocaml-ng.ocamlPackages_5_4.overrideScope (
               oself: osuper: {
                 mdx = osuper.mdx.override {
                   logs = oself.logs;
@@ -63,8 +65,13 @@
           oxcamlOpamRepo = oxcaml-opam-repository;
         };
 
+        # Older nixpkgs for OCaml 4.02 support
+        pkgs-old = nixpkgs-old.legacyPackages.${system}.appendOverlays [
+          ocaml-overlays.overlays.default
+        ];
+
         dune-static-overlay = self: super: {
-          ocamlPackages = super.ocaml-ng.ocamlPackages_5_3.overrideScope (
+          ocamlPackages = super.ocaml-ng.ocamlPackages_5_4.overrideScope (
             oself: osuper: {
               dune_3 = osuper.dune_3.overrideAttrs (a: {
                 src = ./.;
@@ -80,19 +87,16 @@
 
         ocamlformat =
           let
-            ocamlformat_version =
-              let
-                lists = pkgs.lib.lists;
-                strings = pkgs.lib.strings;
-                ocamlformat_config = strings.splitString "\n" (builtins.readFile ./.ocamlformat);
-                prefix = "version=";
-                ocamlformat_version_pred = line: strings.hasPrefix prefix line;
-                version_line = lists.findFirst ocamlformat_version_pred "not_found" ocamlformat_config;
-                version = strings.removePrefix prefix version_line;
-              in
-              builtins.replaceStrings [ "." ] [ "_" ] version;
+            lists = pkgs.lib.lists;
+            strings = pkgs.lib.strings;
+            ocamlformat_config = strings.splitString "\n" (builtins.readFile ./.ocamlformat);
+            prefix = "version=";
+            ocamlformat_version_pred = line: strings.hasPrefix prefix line;
+            version_line = lists.findFirst ocamlformat_version_pred "not_found" ocamlformat_config;
+            version_string = strings.removePrefix prefix version_line;
+            ocamlformat_attr = builtins.replaceStrings [ "." ] [ "_" ] version_string;
           in
-          builtins.getAttr ("ocamlformat_" + ocamlformat_version) pkgs;
+          builtins.getAttr ("ocamlformat_" + ocamlformat_attr) nixpkgs.legacyPackages.${system};
 
         testBuildInputs =
           with pkgs;
@@ -274,13 +278,13 @@
             fmt = pkgs.mkShell {
               inherit INSIDE_NIX;
               nativeBuildInputs = [ ocamlformat ];
-              # re shouldn't be needed. this is an issue with the fmt rules
               inputsFrom = [
                 pkgs.dune_3
               ];
               buildInputs = with pkgs.ocamlPackages; [
                 csexp
                 pp
+                # re shouldn't be needed. this is an issue with the fmt rules
                 re
                 spawn
                 uutf
@@ -342,9 +346,9 @@
 
             bootstrap-check = pkgs.mkShell {
               inherit INSIDE_NIX;
-              buildInputs = with pkgs; [
-                gnumake
-                ocaml-ng.ocamlPackages_4_02.ocaml
+              buildInputs = [
+                pkgs.gnumake
+                pkgs-old.ocaml-ng.ocamlPackages_4_02.ocaml
               ];
               meta.description = ''
                 Provides a minimal shell environment with OCaml 4.02 in order
@@ -354,9 +358,9 @@
 
             bootstrap-check_4_08 = pkgs.mkShell {
               inherit INSIDE_NIX;
-              buildInputs = with pkgs; [
-                gnumake
-                ocaml-ng.ocamlPackages_4_08.ocaml
+              buildInputs = [
+                pkgs.gnumake
+                pkgs-old.ocaml-ng.ocamlPackages_4_08.ocaml
               ];
               meta.description = ''
                 Provides a minimal shell environment with OCaml 4.08 in order

flake.nix

@@ -49,7 +49,7 @@ Test transitive deps can not be directly accessed, both for compiler versions su
   > EOF
 
   $ dune build ./runf.exe 2>&1 | grep -v ocamlc
-  File "runf.ml", line 1, characters 16-21:
+  File "runf.ml", line 1, characters 16-19:
   1 | let _ = Bar.y + Foo.v
-                      ^^^^^
+                      ^^^
   Error: Unbound module Foo

test/blackbox-tests/test-cases/hidden-deps-supported.t/run.t

@@ -17,8 +17,8 @@ We should forbid lib interfaces modules from depending on themselves:
   $ touch bar.ml
 
   $ dune build @check
-  File "foo.ml", line 1, characters 9-14:
+  File "foo.ml", line 1, characters 9-12:
   1 | let () = Foo.f ()
-               ^^^^^
+               ^^^
   Error: Unbound module Foo
   [1]