Skip to content

Conversation

@kit-ty-kate
Copy link
Member

This pull-request concerns:

  • opam-client.2.5.0~rc1: Client library for opam 2.5
  • opam-core.2.5.0~rc1: Core library for opam 2.5
  • opam-devel.2.5.0~rc1: Bootstrapped development binary for opam 2.5
  • opam-format.2.5.0~rc1: Format library for opam 2.5
  • opam-installer.2.5.0~rc1: Installation of files to a prefix, following opam conventions
  • opam-repository.2.5.0~rc1: Repository library for opam 2.5
  • opam-solver.2.5.0~rc1: Solver library for opam 2.5
  • opam-state.2.5.0~rc1: State library for opam 2.5


🐫 Pull-request generated by opam-publish v2.7.0

@jmid
Copy link
Member

jmid commented Nov 20, 2025

Thanks!

There's a puzzling "index out of bounds" triggered by the MSys2 workflow: 🤔

  [ERROR] The compilation of opam-devel.2.5.0~rc1 failed at "make opam-devel.install".
  
  #=== ERROR while compiling opam-devel.2.5.0~rc1 ===============================#
  # context     2.4.1 | win32/x86_64 | ocaml.4.14.2 | file://D:/a/opam-repository/opam-repository
  # path        D:\opamroot\default\.opam-switch\build\opam-devel.2.5.0~rc1
  # command     C:\msys64\usr\bin\make.exe opam-devel.install
  # exit-code   2
  # env-file    D:\opamroot\log\opam-devel-3144-c601b3.env
  # output-file D:\opamroot\log\opam-devel-3144-c601b3.out
  ### output ###
  # dune build  -p opam opam.install
  # File "src/core/dune", lines 62-66, characters 0-200:
  # 62 | (rule
  # 63 |   (enabled_if (= %{os_type} "Win32"))
  # 64 |   (targets opam-putenv.exe)
  # 65 |   (deps    opamInject.c)
  # 66 |   (action  (run ocaml %{dep:../../shell/build-putenv.ml} %{targets} %{dep:opam-putenv.c} %{dep:cc64})))
  # (cd _build/default/src/core && D:\opamroot\default\bin\ocaml.exe ../../shell/build-putenv.ml opam-putenv.exe opam-putenv.c cc64)
  # Exception: Invalid_argument "index out of bounds".
  # make: *** [Makefile:127: opam-devel.install] Error 1

@jmid
Copy link
Member

jmid commented Nov 20, 2025

Hm, it is not new apparently, as it also triggered on the beta1 in #28877

Copy link
Member

@jmid jmid left a comment

Choose a reason for hiding this comment

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

OK, thanks! LGTM 👍

@mseri mseri merged commit 303b095 into ocaml:master Nov 23, 2025
4 checks passed
@mseri
Copy link
Member

mseri commented Nov 23, 2025

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants