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

Updated readme to address potential issue with MacOs #341

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Niminem
Copy link

@Niminem Niminem commented Apr 3, 2024

I banged my head against the wall for a couple of hours trying to get nim/choosenim/etc to work on my mac- I clearly saw ~/.nimble/bin in my path via echo $path but it wouldn't work. Upon further investigation, apparently I didn't have a .zshrc file in my home directory and needed to add the path there.

This was on a clean installation of mac.

It may or may not be common so I figured it's best to add notes to the README.

readme.md Outdated Show resolved Hide resolved
@janAkali
Copy link

janAkali commented Apr 3, 2024

I clearly saw ~/.nimble/bin in my path via echo $path but it wouldn't work.

$path and $PATH are two different things. $PATH is widely used by everything, but I've never seen $path until today.

Co-authored-by: Archar Gelod <60652075+Archargelod@users.noreply.github.com>
@Niminem
Copy link
Author

Niminem commented Apr 3, 2024

@Archargelod Interesting. Oddly enough, both $PATH and $path shown the same for me and figured it was just case-insensitive and the same things. I've accepted those changes from your end. I think we may be good here to finish this one out :)

@Niminem Niminem requested a review from janAkali April 5, 2024 01:16
@Niminem
Copy link
Author

Niminem commented Apr 11, 2024

@Archargelod @dom96 Gentlemen, I would love to make some Youtube videos about Nim and simply point them to Choosenim for installation. Because many of the viewers will not be from a technical background (I do SEO professionally), many will have a Mac and may encounter the issue described above.

Can we merge this PR asap?

@Niminem Niminem closed this Apr 16, 2024
@Niminem Niminem reopened this Apr 16, 2024
@Niminem
Copy link
Author

Niminem commented Apr 30, 2024

@dom96 ?

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.

2 participants