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

Add mkdir as part of .NET core install instructions #517

Merged
merged 2 commits into from Aug 26, 2016
Merged

Add mkdir as part of .NET core install instructions #517

merged 2 commits into from Aug 26, 2016

Conversation

jamesmontemagno
Copy link
Member

From issue: https://github.com/dotnet/cli/issues/3988

Else the rest will fail.

Also add in that you have to have Homebrew installed.

From issue: https://github.com/dotnet/cli/issues/3988

Else the rest will fail.

Also add in that you have to have Homebrew installed.
@bryanmacfarlane
Copy link
Collaborator

hmmm. Didn't need to do.

Do these instructions also need updating?

https://www.microsoft.com/net/core#macos

@jamesmontemagno
Copy link
Member Author

@bryanmacfarlane I would recommend adding it in. Essentially if you don't have a /usr/local/lib/ then it will fail to link. This causes issues with the vsts-agent.

On a fresh machine it does not have it.

I have now tested on 3 machines and 2 of the 3 times it has failed. Others have also ran into this issue. As you can see here: https://github.com/dotnet/cli/issues/3988

@bryanmacfarlane
Copy link
Collaborator

bryanmacfarlane commented Aug 24, 2016

OK - I had it on my fresh machine. hmmm

Put commend above

(pound sign) ensure directory exists

```bash
brew update
brew install openssl
mkdir -p /usr/local/lib/
Copy link
Collaborator

Choose a reason for hiding this comment

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

put a blank line then comment

ensure exists on fresh machine

Copy link
Member Author

Choose a reason for hiding this comment

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

How is that.

@bryanmacfarlane bryanmacfarlane merged commit ce97f69 into microsoft:master Aug 26, 2016
@jamesmontemagno jamesmontemagno deleted the patch-1 branch August 26, 2016 17:34
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.

None yet

2 participants