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

Adding 'install with yarn' to troubleshooting #367

Closed
wants to merge 1 commit into from

Conversation

jondot
Copy link

@jondot jondot commented May 13, 2018

Hi,
Just finished a two-hour session trying to figure out why Reason CLI tools won't install.
Apparently it choked on a path which it thought is too deep inside filesystem, Esy won't be able to relocate binaries.
This specific error was also silent, so I had to go into the esyBuildRelease shell script and make it spit it out.

This issue hinted using Yarn would do off with all that, and it did, installing quickly and happily.

Might also be worth recommending Yarn as a default running for installing over npm, but I will leave that to more knowledgeable documentation editor.

Hi,
Just finished a two-hour session trying to figure out why Reason CLI tools won't install.
Apparently it choked on a path which it thought `is too deep inside filesystem, Esy won't be able to relocate binaries`.
This specific error was also silent, so I had to go into the esyBuildRelease shell script and make it spit it out.

This [issue](reasonml/reason-cli#45) hinted using Yarn would do off with all that, and it did, installing quickly and happily. 

Might also be worth recommending Yarn as a default running for installing over npm, but I will leave that to more knowledgeable documentation editor.
@chenglou
Copy link
Member

cc @andreypopp

@andreypopp
Copy link

I'm not sure that yarn would alway solve this issue — for me it looks like it just happened that yarn installs into a fs location which isn't so deep as the one used by npm.

The issue with swallowing the actual error is the real one — I've pushed a fix into esy/esy#182.

@jondot
Copy link
Author

jondot commented May 23, 2018

Thanks. I can confirm that the difference is where Yarn installs vs npm. But since the documentation states "if this doesn't work, try X, and if not, try Y, ..." and so on, this becomes just another solution to try.

@kevinsimper
Copy link
Contributor

I think this pull request can be closed as yarn is now used in the docs. cc. @chenglou

@chenglou
Copy link
Member

You're right. Sorry for not merging this earlier and thanks again for the contribution @jondot.

@chenglou chenglou closed this Aug 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants