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

Fix YARN global commands #496

Closed
mrbig00 opened this issue Feb 6, 2019 · 3 comments · Fixed by #528
Closed

Fix YARN global commands #496

mrbig00 opened this issue Feb 6, 2019 · 3 comments · Fixed by #528
Assignees

Comments

@mrbig00
Copy link
Contributor

mrbig00 commented Feb 6, 2019

ISSUE TYPE

  • Feature request

I'm using some yarn packages globally, and I've seen that the global packages aren't accessible from command line. Setting the yarn prefix dir and adding it to the PATH fixes the issue

Related SO question

SUMMARY

Add these to the startup script:

su -c 'yarn config set prefix ~/.yarn' -l devilbox
su -c 'export PATH="$PATH:`yarn global bin`"' -l devilbox

Goal

Make yarn global commands accessible

@mrbig00 mrbig00 added the feature label Feb 6, 2019
@mrbig00 mrbig00 changed the title Add yarn global to the PATH var Fix YARN global commands Feb 6, 2019
@cytopia cytopia self-assigned this Feb 28, 2019
@cytopia
Copy link
Owner

cytopia commented Feb 28, 2019

I will rewrite the NodeJS, nvm, yarn, npx installation more properly and will also integrate the above mentioned paths.

There's also another NodeJS related issue open, which might also be interesting/answerable for you: #499

Once I have enough background knowledge regarding this topic (still a NodeJS beginner), I will make this rewrite.


To already have your current issue solved (without the need for rewrite), please use the autostart feature to populate the bash configuration: https://devilbox.readthedocs.io/en/latest/autostart/custom-scripts-globally.html

@cytopia
Copy link
Owner

cytopia commented Mar 8, 2019

@mrbig00 can you try out the branch from the PR to see if this resolves the issue: #528

cytopia added a commit that referenced this issue Mar 9, 2019
@mrbig00
Copy link
Contributor Author

mrbig00 commented Mar 11, 2019

@cytopia I confirm that it works

Great job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants