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.md to clarify installation folder of git version within Mac OS X #4730

Merged
merged 2 commits into from
May 18, 2018

Conversation

tzahari
Copy link
Contributor

@tzahari tzahari commented May 14, 2018

Clarify the root folder of the git installation method for Mac. Thanks to @lukeweston wich was noted this folder in #4726

tzahari and others added 2 commits May 14, 2018 17:43
Clarify the folder of the git installation method for Mac
@d-a-v d-a-v merged commit 83ab034 into esp8266:master May 18, 2018
- Go to Arduino application directory
- For Mac OS X normally:
```bash
cd /Applications/Arduino.app/Contents/Java
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it normal to install Arduino into global applications directory, instead of putting it somewhere in home directory?

Maybe we can relax this to /path/to/Arduino.app?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I used to use mac too some time ago, and that was the place where I installed Arduino, so the place seemed right to me.
But you are right, not everyone is root on his machine, or want to install a single version of arduino in the global app dir.
I will fix this.

Copy link
Contributor Author

@tzahari tzahari May 18, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, you are right. In Mac OS X you can put the app bundle everywhere.
It seems there is no easy way to get the folder of an app via bash.
If you want, I can update the doc as suggested by @igrr

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