Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

Incorrectly creates .bash_profile file on WSL Ubuntu #88

Open
1 task done
cdgriffith opened this issue Sep 9, 2019 · 4 comments
Open
1 task done

Incorrectly creates .bash_profile file on WSL Ubuntu #88

cdgriffith opened this issue Sep 9, 2019 · 4 comments

Comments

@cdgriffith
Copy link

cdgriffith commented Sep 9, 2019

When using the windows subsystem for linux and installing avn it creates a .bash_profile. Doing so makes Ubuntu no longer load the .profile file and therefor no longer loads the .bashrc file.

Top of ~/.profile

# ~/.profile: executed by the command interpreter for login shells.
# This file is not read by bash(1), if ~/.bash_profile or ~/.bash_login
# exists.

By default nvm is loaded into bashrc and I believe avn should be added below it.

Details

  • avn 0.2.4
  • node 10.16.3
  • nvm 0.34.0
  • bash 4.4.19

avn is loaded in my ~/.bash_profile file with:

[[ -s "$HOME/.avn/bin/avn.sh" ]] && source "$HOME/.avn/bin/avn.sh" # load avn

nvm specific

  • As an nvm user I am confirming that I did not install with Homebrew
@wbyoung
Copy link
Owner

wbyoung commented Sep 12, 2019

Can you find an example of another tool that does a better job with figuring out which script file to use based on platform? Something like RVM may be a good place to look. Then we can model after work that's already been done.

@ljharb
Copy link

ljharb commented Sep 12, 2019

I’d be interested in a reliable algorithm for nvm’s own install script; I’ve yet to find one.

@evenfrost
Copy link

There's open PR for that: #51.

@musahaidari
Copy link

Just to mention the obvious, till the time the PR #51 is ready, WSL users can delete ~/.bash_profile and add [[ -s "$HOME/.avn/bin/avn.sh" ]] && source "$HOME/.avn/bin/avn.sh" # load avn to the end of ~/.profile.

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

No branches or pull requests

5 participants