Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TypeError: arg.includes is not a function when trying to get tests running with OCaml 5.2.0 #792

Closed
polytypic opened this issue May 15, 2024 · 2 comments
Labels
invalid This doesn't seem right

Comments

@polytypic
Copy link

polytypic commented May 15, 2024

I'm trying to get tests running on 5.2.0 on Windows, see ocaml-multicore/picos#104.

I was pointed to ocaml/opam-repository#25861 and to use the source of the PR https://github.com/dra27/opam-repository.git#windows-initial as the opam repository.

It seems that doing so makes setup-ocaml run into an error:

  D:\a\_actions\ocaml\setup-ocaml\v2\dist\index.cjs:19591
          if (!arg.includes(" ") && !arg.includes("	") && !arg.includes('"')) {
                   ^
  
  TypeError: arg.includes is not a function
      at ToolRunner._uvQuoteCmdArg (D:\a\_actions\ocaml\setup-ocaml\v2\dist\index.cjs:19591:18)
      at ToolRunner._windowsQuoteCmdArg (D:\a\_actions\ocaml\setup-ocaml\v2\dist\index.cjs:19532:23)
      at ToolRunner._getCommandString (D:\a\_actions\ocaml\setup-ocaml\v2\dist\index.cjs:19474:31)
      at ToolRunner.<anonymous> (D:\a\_actions\ocaml\setup-ocaml\v2\dist\index.cjs:19662:51)
      at Generator.next (<anonymous>)
      at D:\a\_actions\ocaml\setup-ocaml\v2\dist\index.cjs:19428:71
      at new Promise (<anonymous>)
      at __awaiter (D:\a\_actions\ocaml\setup-ocaml\v2\dist\index.cjs:19410:14)
      at D:\a\_actions\ocaml\setup-ocaml\v2\dist\index.cjs:19654:52
      at new Promise (<anonymous>)
  
  Node.js v20.8.1

I don't know what causes this. This could be a bug in setup-ocaml (at least not checking/validating/parsing some inputs/data properly into type safe data structures) or something wrong with the #windows-initial repository/branch.

@kit-ty-kate
Copy link
Member

@kit-ty-kate
Copy link
Member

although it feels to me like a better error message could be added

@smorimoto smorimoto added the invalid This doesn't seem right label Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

3 participants