ocaml / ocaml/dune

Sanitize paths outside of Opam’s prefix

Open
#4,541 0 comments 0 reactions 1 assignee View on GitHub

@voodoos is already working on this.

Since Apr 29, 2021.

macos test-suite
Dominant language
OCaml
Stars
1.9k
Forks
500
Avg merge
15h 21m
Merged PRs (30d)
277

Description

Expected Behaviour

Tests should pass irrespective of the paths due to system compiler installation.

Actual Behavior

Tests fail because of unsanitized paths due system compiler installation.

Specifications

  • Version of dune 2.8.5:
  • Version of ocaml 4.10.0
  • Operating system macOS Big Sur:

Additional information

While hacking on dune, I happened to have failing tests. From dune-dev channel, @voodoos hinted that it was because only paths inside opam’s prefix, that is for non-system compiler installation were being sanitised.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.