`$ dune install` shouldn't install to opam prefix
Open
Nobody has claimed this yet.
build
enhancement
package management
- Dominant language
- OCaml
- Stars
- 1.9k
- Forks
- 500
- Avg merge
- 15h 21m
- Merged PRs (30d)
- 277
Description
$ dune install is currently defaulting to install in the opam prefix. This is problematic because it's modifying the switch state without updating the switch state.
There's a few ways to address this problem:
- Warn the user when this is happening. This is the least we can do for now.
- In dune 4.0, stop install to the opam prefix completely. There's really no need for it in the first place. If the user has opam installed, they also have
opam-installerwhich can do the same thing. dune installcan useopam-installerunder the hood when installing to the opam switch
cc @rjbou
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue names no files or tests. Start by tracing the $ dune install entry point and how it selects the opam prefix; review the discussion of warning, removal, or opam-installer integration, then define tests for the chosen behavior and verify that the switch state is not changed silently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ocaml
- Domain
- build-system
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 30/100