Comments (5)
The ocamlfind in reason-cli isn't meant to be used for anything except debugging reason-cli at the moment. OCamlfind needs to know where stuff is - and so it needs its env variables / config populated correctly. I think that every project should have its own ocamlfind dependency, and its own config that is populated correctly, for that specific project, which will avoid the problem.
OPAM does this for you at the global level which causes some problems (much like the problem you are seeing here). Esy (which reason-cli is based off of) does it at the project level, for your project's specific set of dependencies.
I think we should stop exposing ocamlfind from reason-cli or at least discourage it for anything other than debugging a reason-cli installation. Thoughts?
from reason-cli.
@jordwalke Is there any way to call it something other than ocamlfind so that I don't end up accidentally using it? I want to make sure I have a correct opam install so that things don't sometimes work and sometimes not.
If you think we could get away with not shipping it then that solves my problem :)
from reason-cli.
Yeah, we can realias binaries that are only coincidentally exported to something like reason-cli-ocamlfind
. That way they can still be used to debug the installation, but won't conflict.
from reason-cli.
Hey sorry I didn't comment on this for a while.
It'd be awesome if we could make this happen
from reason-cli.
Heyyyy this hit us again, where the ocamlfind that comes with Reason-cli shadows opam’s ocamlfind which makes it unusable.
Any chance we can get this fixed quickly?
from reason-cli.
Related Issues (20)
- New release? HOT 5
- rtop not working on arch linux
- Get this working with merlin-lsp
- Hi there, can we archive this repo and add instructions to use bsrefmt from bs-platform? HOT 1
- Offer option to keep semicolons removed with refmt
- Refmt with deriving abstract
- refmt should move ternary operators onto the next line as the first non-whitespace char HOT 1
- ocamlmerlin binary broken in 3.3.3-macos? HOT 9
- An option for refmt line length HOT 1
- `npm i -g reason-cli` install macOS version on linux HOT 3
- CI option for refmt to check only HOT 1
- Fastpipe operations in JSX are modified to invalid code after run through refmt
- postinstall fails in openSUSE Tumbleweed
- Whitespace with new fast pipes?
- Reason cli global installation by yarn vs npm HOT 1
- Refmt 3.3.9 removes semi-colon in JSX
- npm install fails on MacOS Mojave
- Adding reason-cli as a project development dependency
- Incorrect conversion of custom fat arrow operator HOT 2
- refmt doesnt handle "//" comment syntax HOT 5
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from reason-cli.