Skip to content
This repository has been archived by the owner on Nov 13, 2021. It is now read-only.

Add build instructions for OS X 10.11 users #110

Merged
2 commits merged into from
Sep 18, 2016
Merged

Conversation

schultyy
Copy link
Member

@schultyy schultyy commented Sep 18, 2016

Resolves #100
On OS X 10.11 the build of semantic-rs fails because of missing OpenSSL headers. To prevent the build from failure, some environment variables need to be configured.

On OS X 10.11 the build of semantic-rs fails because of missing OpenSSL headers. To prevent the build from failure, some environment variables need to be configured.
Copy link
Member

@badboy badboy left a comment

Choose a reason for hiding this comment

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

Are really both variables necessary?

Copy link
Member

@badboy badboy left a comment

Choose a reason for hiding this comment

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

👍

@badboy
Copy link
Member

badboy commented Sep 18, 2016

@not-a-robot r+

@badboy badboy closed this Sep 18, 2016
@badboy badboy reopened this Sep 18, 2016
@badboy
Copy link
Member

badboy commented Sep 18, 2016

@not-a-robot r+

@ghost
Copy link

ghost commented Sep 18, 2016

📌 Commit 6d40de6 has been approved by badboy

@ghost
Copy link

ghost commented Sep 18, 2016

⌛ Testing commit 6d40de6 with merge 6425ea0...

ghost pushed a commit that referenced this pull request Sep 18, 2016
Add build instructions for OS X 10.11 users

Resolves #100
On OS X 10.11 the build of semantic-rs fails because of missing OpenSSL headers. To prevent the build from failure, some environment variables need to be configured.
@ghost
Copy link

ghost commented Sep 18, 2016

☀️ Test successful - status-travis
Approved by: badboy
Pushing 6425ea0 to master...

@ghost ghost merged commit 6d40de6 into master Sep 18, 2016
@schultyy schultyy deleted the fix-osx-10-11-build-errors branch December 2, 2016 21:40
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants