2026-05-13 10:19.00: New job: test ocsigen/ocsigen-dune-rules https://github.com/ocsigen/ocsigen-dune-rules.git#refs/heads/modernize (ab4d0c049fd6b941081f57b76a3f078b08ac62bd) (linux-x86_64:(lint-fmt)) Base: ocaml/opam:debian-13-ocaml-4.11@sha256:6c34153b2b07fc0f884e792a60178e250c2d06a45050e01ed0883a67ec11440c ocamlformat version: version 0.28.1 (from opam) To reproduce locally: git clone --recursive "https://github.com/ocsigen/ocsigen-dune-rules.git" -b "modernize" && cd "ocsigen-dune-rules" && git reset --hard ab4d0c04 cat > Dockerfile <<'END-OF-DOCKERFILE' FROM ocaml/opam:debian-13-ocaml-4.11@sha256:6c34153b2b07fc0f884e792a60178e250c2d06a45050e01ed0883a67ec11440c USER 1000:1000 RUN cd ~/opam-repository && (git cat-file -e a4f153d7481c469b20f3b1064338d6a21abcb832 || git fetch origin master) && git reset -q --hard a4f153d7481c469b20f3b1064338d6a21abcb832 && git log --no-decorate -n1 --oneline && opam update -u RUN opam depext -i dune WORKDIR /src RUN opam depext -i ocamlformat=0.28.1 COPY --chown=1000:1000 . /src/ RUN opam exec -- dune build @fmt --ignore-promoted-rules || (echo "dune build @fmt failed"; exit 2) END-OF-DOCKERFILE docker build . END-REPRO-BLOCK 2026-05-13 10:19.00: Using cache hint "ocsigen/ocsigen-dune-rules-ocaml/opam:debian-13-ocaml-4.11@sha256:6c34153b2b07fc0f884e792a60178e250c2d06a45050e01ed0883a67ec11440c-debian-13-4.11_opam-2.5-ocamlformat-a4f153d7481c469b20f3b1064338d6a21abcb832" 2026-05-13 10:19.00: Using OBuilder spec: ((from ocaml/opam:debian-13-ocaml-4.11@sha256:6c34153b2b07fc0f884e792a60178e250c2d06a45050e01ed0883a67ec11440c) (user (uid 1000) (gid 1000)) (run (cache (opam-archives (target /home/opam/.opam/download-cache))) (network host) (shell "cd ~/opam-repository && (git cat-file -e a4f153d7481c469b20f3b1064338d6a21abcb832 || git fetch origin master) && git reset -q --hard a4f153d7481c469b20f3b1064338d6a21abcb832 && git log --no-decorate -n1 --oneline && opam update -u")) (run (cache (opam-archives (target /home/opam/.opam/download-cache))) (network host) (shell "opam depext -i dune")) (workdir /src) (run (cache (opam-archives (target /home/opam/.opam/download-cache))) (network host) (shell "opam depext -i ocamlformat=0.28.1")) (copy (src .) (dst /src/)) (run (shell "opam exec -- dune build @fmt --ignore-promoted-rules || (echo \"dune build @fmt failed\"; exit 2)")) ) 2026-05-13 10:19.00: Waiting for resource in pool OCluster 2026-05-13 10:40.18: Waiting for worker… 2026-05-13 10:42.27: Got resource from pool OCluster Building on toxis.caelum.ci.dev All commits already cached HEAD is now at ab4d0c0 test: cram coverage for the three new gen options (from ocaml/opam:debian-13-ocaml-4.11@sha256:6c34153b2b07fc0f884e792a60178e250c2d06a45050e01ed0883a67ec11440c) 2026-05-13 10:42.21 ---> using "36abb6db1723167d15c41f53ef05d371444812dc64e2ea7fe109d222db988473" from cache /: (user (uid 1000) (gid 1000)) /: (run (cache (opam-archives (target /home/opam/.opam/download-cache))) (network host) (shell "cd ~/opam-repository && (git cat-file -e a4f153d7481c469b20f3b1064338d6a21abcb832 || git fetch origin master) && git reset -q --hard a4f153d7481c469b20f3b1064338d6a21abcb832 && git log --no-decorate -n1 --oneline && opam update -u")) From https://github.com/ocaml/opam-repository * branch master -> FETCH_HEAD fc08333d1b..ba03103280 master -> origin/master a4f153d748 Merge pull request #29889 from mtelvers/opam-publish-ocaml-version.4.1.1 <><> Updating package repositories ><><><><><><><><><><><><><><><><><><><><><><> [default] synchronised from git+file:///home/opam/opam-repository Everything as up-to-date as possible (run with --verbose to show unavailable upgrades). The following packages are not being upgraded because the new versions conflict with other installed packages: - ocaml.5.6.0 - ocaml-config.3 However, you may "opam upgrade" these packages explicitly, which will ask permission to downgrade or uninstall the conflicting packages. Nothing to do. # Run eval $(opam env) to update the current shell environment 2026-05-13 10:42.21 ---> using "57933430ce6ee04fd567f281e7ebe5f3ade1880481602bde6f454d0e84e43b4d" from cache /: (run (cache (opam-archives (target /home/opam/.opam/download-cache))) (network host) (shell "opam depext -i dune")) Since version 2.1, opam now handles external dependencies alongside OCaml ones, and the `depext` plugin interface is provided for backwards compatibility only. Consider using your usual `opam install` command to install both OCaml and system dependencies, or `opam install <pkg> --depext-only` if you want to only install external dependencies. # Detecting depexts using vars: arch=x86_64, os=linux, os-distribution=debian, os-family=debian # No extra OS packages requirements found. The following actions will be performed: - install dune 3.22.2 <><> Processing actions <><><><><><><><><><><><><><><><><><><><><><><><><><><><> -> retrieved dune.3.22.2 (cached) -> installed dune.3.22.2 Done. # Run eval $(opam env) to update the current shell environment 2026-05-13 10:42.21 ---> using "0660a7a1aacf9713843c7c8443a91791ddff911339fa457c1ce83edfc713c9b0" from cache /: (workdir /src) /src: (run (cache (opam-archives (target /home/opam/.opam/download-cache))) (network host) (shell "opam depext -i ocamlformat=0.28.1")) Since version 2.1, opam now handles external dependencies alongside OCaml ones, and the `depext` plugin interface is provided for backwards compatibility only. Consider using your usual `opam install` command to install both OCaml and system dependencies, or `opam install <pkg> --depext-only` if you want to only install external dependencies. # Detecting depexts using vars: arch=x86_64, os=linux, os-distribution=debian, os-family=debian # No extra OS packages requirements found. The following actions will be performed: - install ocamlbuild 0.16.1 [required by fpath, astring, uuseg] - install either 1.0.0 [required by ocamlformat-lib] - install menhirLib 20260209 [required by ocamlformat-lib] - install csexp 1.5.2 [required by ocamlformat] - install menhirSdk 20260209 [required by ocamlformat-lib] - install menhirGLR 20260209 [required by menhir] - install cmdliner 2.1.1 [required by ocamlformat] - install camlp-streams 5.0.1 [required by ocamlformat-lib] - install dune-build-info 3.22.2 [required by ocamlformat-lib] - install seq base [required by re] - install ocaml-version 4.1.1 [required by ocamlformat-lib] - install fix 20250919 [required by ocamlformat-lib] - install sexplib0 v0.15.1 [required by base] - install ocamlfind 1.9.8 [required by ocp-indent, astring, fpath, uuseg] - install menhirCST 20260209 [required by menhir] - install dune-configurator 3.22.2 [required by base] - install re 1.11.0 [required by ocamlformat] - install topkg 1.1.1 [required by fpath, astring, uuseg] - install ocp-indent 1.9.0 [required by ocamlformat-lib] - install menhir 20260209 [required by ocamlformat-lib] - install base v0.15.2 [required by ocamlformat-lib] - install uutf 1.0.4 [required by ocamlformat-lib] - install astring 0.8.5 [required by ocamlformat-lib] - install stdio v0.15.0 [required by ocamlformat-lib] - install uucp 15.0.0 [required by uuseg] - install fpath 0.7.3 [required by ocamlformat-lib] - install uuseg 15.0.0 [required by ocamlformat-lib] - install ocamlformat-lib 0.28.1 [required by ocamlformat] - install ocamlformat 0.28.1 ===== 29 to install ===== <><> Processing actions <><><><><><><><><><><><><><><><><><><><><><><><><><><><> -> retrieved astring.0.8.5 (cached) -> retrieved camlp-streams.5.0.1 (cached) -> retrieved base.v0.15.2 (cached) -> retrieved cmdliner.2.1.1 (cached) -> retrieved csexp.1.5.2 (cached) -> retrieved either.1.0.0 (cached) -> retrieved fix.20250919 (cached) -> retrieved fpath.0.7.3 (cached) -> installed camlp-streams.5.0.1 -> retrieved menhir.20260209 (cached) -> retrieved menhirCST.20260209 (cached) -> installed csexp.1.5.2 -> installed either.1.0.0 -> retrieved menhirGLR.20260209 (cached) -> retrieved menhirLib.20260209 (cached) -> installed fix.20250919 -> installed menhirCST.20260209 -> retrieved menhirSdk.20260209 (cached) -> retrieved ocaml-version.4.1.1 (cached) -> retrieved ocamlbuild.0.16.1 (cached) -> retrieved ocamlfind.1.9.8 (cached) -> retrieved dune-configurator.3.22.2 (cached) -> retrieved dune-build-info.3.22.2 (cached) -> installed menhirGLR.20260209 -> installed menhirLib.20260209 -> installed menhirSdk.20260209 -> installed ocaml-version.4.1.1 -> retrieved ocamlformat.0.28.1 (cached) -> retrieved ocamlformat-lib.0.28.1 (cached) -> retrieved seq.base (cached) -> installed seq.base -> retrieved ocp-indent.1.9.0 (cached) -> retrieved re.1.11.0 (cached) -> retrieved sexplib0.v0.15.1 (cached) -> retrieved stdio.v0.15.0 (cached) -> retrieved topkg.1.1.1 (cached) -> retrieved uuseg.15.0.0 (cached) -> retrieved uutf.1.0.4 (cached) -> installed dune-build-info.3.22.2 -> retrieved uucp.15.0.0 (cached) -> installed dune-configurator.3.22.2 -> installed sexplib0.v0.15.1 -> installed cmdliner.2.1.1 -> installed re.1.11.0 -> installed ocamlfind.1.9.8 -> installed ocamlbuild.0.16.1 -> installed ocp-indent.1.9.0 -> installed base.v0.15.2 -> installed stdio.v0.15.0 -> installed topkg.1.1.1 -> installed menhir.20260209 -> installed uutf.1.0.4 -> installed astring.0.8.5 -> installed fpath.0.7.3 -> installed uucp.15.0.0 -> installed uuseg.15.0.0 -> installed ocamlformat-lib.0.28.1 -> installed ocamlformat.0.28.1 Done. <><> ocp-indent.1.9.0 installed successfully ><><><><><><><><><><><><><><><><><> => This package requires additional configuration for use in editors. Install package 'user-setup', or manually: * for Emacs, add these lines to ~/.emacs: (add-to-list 'load-path "/home/opam/.opam/4.11/share/emacs/site-lisp") (require 'ocp-indent) * for Vim, add this line to ~/.vimrc: set rtp^="/home/opam/.opam/4.11/share/ocp-indent/vim" # Run eval $(opam env) to update the current shell environment 2026-05-13 10:43.58 ---> saved as "ed8c08b79112e24a23433f12b91916ca9777d4a776fee504e16bea48f654b197" /src: (copy (src .) (dst /src/)) 2026-05-13 10:43.59 ---> saved as "d8db20c5d7026568cb2345cb42f2f99758ba10f916fbdf45200d6add32cf1c5c" /src: (run (shell "opam exec -- dune build @fmt --ignore-promoted-rules || (echo \"dune build @fmt failed\"; exit 2)")) File "ocsigen-dune-rules/main.ml", line 1, characters 0-0: diff --git a/_build/default/ocsigen-dune-rules/main.ml b/_build/default/ocsigen-dune-rules/.formatted/main.ml index a25a214..fd07665 100644 --- a/_build/default/ocsigen-dune-rules/main.ml +++ b/_build/default/ocsigen-dune-rules/.formatted/main.ml @@ -5,9 +5,7 @@ module Gen = struct (match internal_prefix with | Some p -> Gen_rules.extra_ppx_args := [ "-internal-prefix"; p ] | None -> ()); - (match subdir with - | Some s -> Gen_rules.subdir_name := s - | None -> ()); + (match subdir with Some s -> Gen_rules.subdir_name := s | None -> ()); (match server_objs_dir with | Some d -> Gen_rules.server_objs_dir := d | None -> ()); @@ -22,11 +20,11 @@ module Gen = struct let arg_internal_prefix = let doc = "Pass [-internal-prefix $(docv)] to ocsigen-ppx-client. Tells the \ - client PPX to strip the [$(docv)__] wrapper prefix from the type \ - paths it reads in the server [.cmo] files, so that the generated \ - client code references the user-visible names instead of the \ - internal ones. Required when compiling a wrapped library whose \ - [%client] blocks refer to its own modules (e.g. ocsigen-start)." + client PPX to strip the [$(docv)__] wrapper prefix from the type paths \ + it reads in the server [.cmo] files, so that the generated client code \ + references the user-visible names instead of the internal ones. \ + Required when compiling a wrapped library whose [%client] blocks refer \ + to its own modules (e.g. ocsigen-start)." in Arg.( value @@ -37,21 +35,20 @@ module Gen = struct let doc = "Wrap the generated rules in a [(subdir $(docv) ...)] stanza so the \ preprocessed files land in [$(docv)/]. Used together with \ - [(include_subdirs qualified)] to expose the modules under a \ - [$(docv).] namespace." + [(include_subdirs qualified)] to expose the modules under a [$(docv).] \ + namespace." in Arg.(value & opt (some string) None & info ~doc ~docv:"DIR" [ "subdir" ]) let arg_server_objs_dir = let doc = - "Path to the server library's [.objs/byte/] directory, relative to \ - the dune file containing the generated rules. When set, emit \ - explicit [%{dep:$(docv)/<prefix>__<Name>.cmo}] paths for \ - [-server-cmo] instead of the [%{cmo:Name}] dune variable. Needed \ - when the client lib has a sister module of the same name as the \ - server, in which case [%{cmo:Name}] resolves to the local (client) \ - [.cmo] rather than the server's. The [<prefix>__] is derived from \ - [--subdir]." + "Path to the server library's [.objs/byte/] directory, relative to the \ + dune file containing the generated rules. When set, emit explicit \ + [%{dep:$(docv)/<prefix>__<Name>.cmo}] paths for [-server-cmo] instead \ + of the [%{cmo:Name}] dune variable. Needed when the client lib has a \ + sister module of the same name as the server, in which case \ + [%{cmo:Name}] resolves to the local (client) [.cmo] rather than the \ + server's. The [<prefix>__] is derived from [--subdir]." in Arg.( value @@ -61,10 +58,7 @@ module Gen = struct let cmd = let term = Term.( - const run - $ arg_internal_prefix - $ arg_subdir - $ arg_server_objs_dir + const run $ arg_internal_prefix $ arg_subdir $ arg_server_objs_dir $ arg_dir) in let doc = "Generate dune rules to stdout." in File "ocsigen-dune-rules/gen_rules.ml", line 1, characters 0-0: diff --git a/_build/default/ocsigen-dune-rules/gen_rules.ml b/_build/default/ocsigen-dune-rules/.formatted/gen_rules.ml index 6e04ff3..6a7aa5f 100644 --- a/_build/default/ocsigen-dune-rules/gen_rules.ml +++ b/_build/default/ocsigen-dune-rules/.formatted/gen_rules.ml @@ -1,28 +1,27 @@ let pf = Printf.printf let spf = Printf.sprintf -(** Extra arguments to pass to [ocsigen-ppx-client] for every rule. - Currently used by [--internal-prefix]. *) +(** Extra arguments to pass to [ocsigen-ppx-client] for every rule. Currently + used by [--internal-prefix]. *) let extra_ppx_args = ref [] -(** When non-empty, wrap generated rules in a [(subdir DIR ...)] stanza - so the preprocessed files land in [DIR/]. Used together with +(** When non-empty, wrap generated rules in a [(subdir DIR ...)] stanza so the + preprocessed files land in [DIR/]. Used together with [(include_subdirs qualified)]. *) let subdir_name = ref "" -(** When non-empty, emit explicit [%{dep:DIR/<prefix>__<Name>.cmo}] - paths for [-server-cmo] instead of the [%{cmo:Name}] dune - variable. Needed when the client lib has a sister module of the - same name as the server, in which case [%{cmo:Name}] resolves to - the local (client) [.cmo] rather than the server's. *) +(** When non-empty, emit explicit [%{dep:DIR/<prefix>__<Name>.cmo}] paths for + [-server-cmo] instead of the [%{cmo:Name}] dune variable. Needed when the + client lib has a sister module of the same name as the server, in which case + [%{cmo:Name}] resolves to the local (client) [.cmo] rather than the + server's. *) let server_objs_dir = ref "" let gen_eliom_ppx_rule ~target ~input ~args = (* The [chdir] instruction is needed to obtain the correct path for [-loc-filename] to be used in error messages. *) let all_args = !extra_ppx_args @ args in - if !subdir_name <> "" - then + if !subdir_name <> "" then pf {|(subdir %s (rule @@ -40,34 +39,34 @@ let gen_eliom_ppx_rule ~target ~input ~args = (chdir %%{workspace_root} (run ocsigen-ppx-client -as-pp -loc-filename %%{dep:%s} %s %%{dep:%s})))) |} - target input (String.concat " " all_args) input + target input + (String.concat " " all_args) + input (** Compute the [-server-cmo] argument for a given module file. - Default: rely on the [%{cmo:...}] dune variable to find the server - [.cmo] in the current library. + Default: rely on the [%{cmo:...}] dune variable to find the server [.cmo] in + the current library. - With [--server-objs-dir DIR]: build an explicit [%{dep:...}] path - pointing at [DIR/<prefix>__<Name>.cmo], where [<prefix>__] is - derived from [--subdir] (the lowercase of the subdir name plus - [__], to match dune's own wrapping convention). This avoids the - ambiguity of [%{cmo:Name}] when client and server libs both have a - [Name] module. *) + With [--server-objs-dir DIR]: build an explicit [%{dep:...}] path pointing + at [DIR/<prefix>__<Name>.cmo], where [<prefix>__] is derived from [--subdir] + (the lowercase of the subdir name plus [__], to match dune's own wrapping + convention). This avoids the ambiguity of [%{cmo:Name}] when client and + server libs both have a [Name] module. *) let server_cmo_arg ~server_rel_prefix ~fname_no_ext = - if !server_objs_dir <> "" - then + if !server_objs_dir <> "" then let module_base = Filename.basename fname_no_ext in let cap_name = String.capitalize_ascii module_base in let prefix = - if !subdir_name <> "" - then String.lowercase_ascii !subdir_name ^ "__" + if !subdir_name <> "" then String.lowercase_ascii !subdir_name ^ "__" else "" in let cmo_from_dune_dir = spf "%s/%s%s.cmo" !server_objs_dir prefix cap_name in let cmo_path = - if !subdir_name <> "" then spf "../%s" cmo_from_dune_dir else cmo_from_dune_dir + if !subdir_name <> "" then spf "../%s" cmo_from_dune_dir + else cmo_from_dune_dir in spf "%%{dep:%s}" cmo_path else dune build @fmt failed "/usr/bin/env" "bash" "-c" "opam exec -- dune build @fmt --ignore-promoted-rules || (echo "dune build @fmt failed"; exit 2)" failed with exit status 2 2026-05-13 10:44.13: Job failed: Failed: Build failed