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.

Sign up for GitHub

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

Jump to bottom

Possible issue detecting older choosenim install #264

Open
circular opened this issue Sep 5, 2021 · 4 comments · May be fixed by #279
Open

Possible issue detecting older choosenim install #264

circular opened this issue Sep 5, 2021 · 4 comments · May be fixed by #279
Labels
Bug Hacktoberfest

Comments

@circular
Copy link

circular commented Sep 5, 2021 •
edited

Running choosenim using the curl command installed the older stable version (1.2.0 instead of 1.4.8).

Perhaps since there was an older choosenim install already, the script was somehow unable to detect the older install.

This was resolved for now by running choosenim update stable, however the author requested that an issue be filed to look into the older-install check.
@dom96
Copy link
Owner

dom96 commented Oct 2, 2021

I think this might be an issue with this script in case anyone wants to have a look: https://github.com/dom96/choosenim/blob/master/scripts/choosenim-unix-init.sh

@heinthanth
Copy link

@circular Let me clear your problem ...

Did you mean:

  • You have installed choosenim by downloading from Github release page and then, install nim 1.2.0
  • Then, you run choosenim installation script?

@dom96 How do you understand his problem ... Am I right?

@dom96
Copy link
Owner

dom96 commented Oct 4, 2021

My guess is:

  • Choosenim was installed using the script, it downloaded Nim 1.2.0
  • After a few months later the script was run again, it once again installed Nim 1.2.0 which was unexpected

A fix here could be as simple as running choosenim update stable automatically in the script.

@ganderzz
Copy link

If this is up for grabs, I can try taking a look at it. :) I'm guessing it's as simple as adding @dom96 's suggestion after downloading choosenim?

@ganderzz ganderzz linked a pull request Oct 16, 2021 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: run choosenim update stable before downloading nim ganderzz/choosenim
4 participants
@circular @dom96 @ganderzz @heinthanth