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

Adds bin folder creation instruction #6846

Closed
wants to merge 1 commit into from
Closed

Adds bin folder creation instruction #6846

wants to merge 1 commit into from

Conversation

joelrfcosta
Copy link
Contributor

On some cases, after OS installation, the user/local/bin folder does not exists. Adding this instruction will avoid curl issues.

@joelrfcosta
Adds bin folder creation instruction
fd4a0b9 
On some cases, after OS installation, the user/local/bin folder does not exists. Adding this instruction will avoid curl issues.
@wouterj
Copy link
Member

wouterj commented Aug 5, 2016

Seems correct to me.

👍

status: reviewed

@javiereguiluz
Copy link
Member

I'm not sure about this change because I don't know the prevalence of this error. Is that common to have /usr/local/bin/ missing?

@wouterj
Copy link
Member

wouterj commented Aug 5, 2016

Seems like this is normal on OSx PCs:

What Gary suspects is true. Prior to Tiger the /usr/local directory was created during installation of the OS. But beginning with Tiger it is not. I'm not sure why this change was made but I've run into at least one software installer that failed on Tiger because it expected /usr/local to already be present.
-- https://discussions.apple.com/thread/616992?start=0&tstart=0

/usr/local does not exist on a standard plain OS X install
-- http://apple.stackexchange.com/a/157981

@joelrfcosta
Copy link
Contributor Author

Just like @wouterj said, on newer OS versions this was removed. Usually this is not noticed because we, as developers, already installed something in that folder.

@weaverryan
Copy link
Member

Thanks Joel! I'll admit, it's unfortunate that we need this, but I really appreciate the research you guys did, and this solution makes perfect sense. Cheers!

weaverryan added a commit that referenced this pull request Aug 21, 2016
@weaverryan
minor #6846 Adds bin folder creation instruction (joelrfcosta)
75d6c1c 
This PR was submitted for the 3.1 branch but it was merged into the 2.7 branch instead (closes #6846).

Discussion
----------

Adds bin folder creation instruction

On some cases, after OS installation, the user/local/bin folder does not exists. Adding this instruction will avoid curl issues.

Commits
-------

65a5025 Adds bin folder creation instruction
@weaverryan weaverryan closed this Aug 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Status: Reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@joelrfcosta @wouterj @javiereguiluz @weaverryan @carsonbot