If dune tools install fails to install a tool it can break a user's environment

[sroerick]

Expected Behavior

lsptools are installed

Actual Behavior

Installer fails on 'dune tools install ocamllsp, with message

File "dev-tools.locks/ocaml-lsp-server/ocaml-compiler.pkg", line 112, characters 3-76:
112 |    https://github.com/ocaml/ocaml/releases/download/5.4.0/ocaml-5.4.0.tar.gz)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Error: failed to extract 'download'
Reason: 'tar' failed with non-zero exit code '1' and output:
- tar: input compressed with gzip; use the -z option to decompress it.

I can no longer dune build, there seems to be no way to uninstall the tool, and the incomplete install has prevented all other progress on this repo.

Reproduction

I think this is derived from the OpenBSD version of tar? Other 'dune tools install' command fail in the same way

Specifications

• Version of dune (output of dune --version):
  3.20.2
• Version of ocaml (output of ocamlc --version):
• Operating system (distribution and version):
• OpenBSD 7.8

[shonfeder]

Thank you for the report, @sroerick, and I'm sorry you had to wait so long for a proper reply. We don't currently have support for dune package management outside of linux and MacOS, unfortunately. This is on the roadmap, however, and the limitation is an important part of why the features are stilled marked as 'experimental'. Your report will help us motivate that support, which we know to be important.

[sroerick]

No problem! Thanks. I was able to continue by deleting the .dunetools dir.

I love OpenBSD and do a lot of dev and deployment there, but will stick to linux for first class DX.

Hopefully a pretty easy fix, maybe I can contribute when I get a bit more comfortable. Thanks

[shonfeder]

Contributions welcome :) Fortunately @Alizter seems to have a fix in the works here.

This issue falls into two parts:

1. dune tools must have better error handling: it cannot leave user environments broken because a package fails to install (for whatever reason)
2. We need to support tar correctly on OpenBSD. This is tracked in tar on OpenBSD #10123

I will retitle this issue to reflect the general problem of (1).

[Alizter]

@sroerick Could you test out #12917

[shonfeder]

@sroerick if you have time to test #12917 , as @Alizter proposes, you can install his branch of dune into your opam switch with

opam pin dune git@github.com:Alizter/dune.git#push-tkrvytnyslnk

Please don't feel an obligation to test this, but we would be grateful for any feedback you have time to share!