Skip to content

troubleshooting.md: use /usr/local/opt/openssl for link source #308

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

Merged
merged 1 commit into from
Dec 7, 2016

Conversation

cielavenir
Copy link
Contributor

No description provided.

@msftclas
Copy link

msftclas commented Oct 3, 2016

Hi @cielavenir, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@daviwil
Copy link
Contributor

daviwil commented Oct 3, 2016

Thanks @cielavenir! I don't have my MacBook handy, is the new path a symbolic link which Homebrew creates so that we don't need to point to a specific install of OpenSSL?

@cielavenir
Copy link
Contributor Author

Uh, actually I'm new to Homebrew... (I just installed VSCode extension and followed the information link to come here)
[It looks like] /usr/local/Library/LinkedKegs used to be used as versioning-free space, but it is now moved to /usr/local/opt (according to https://github.com/Homebrew/brew/blob/master/Library/Homebrew/utils.rb#L547 ).

@daviwil
Copy link
Contributor

daviwil commented Dec 7, 2016

@cielavenir thanks again for sending this PR. I finally got a chance to verify this path on my MacBook and it seems that this path works! Thanks a lot for locating this fix, it should help people avoid changing their link for future OpenSSL versions.

@daviwil daviwil merged commit b8a06ab into PowerShell:master Dec 7, 2016
@daviwil daviwil mentioned this pull request Dec 7, 2016
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.

3 participants