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

travis: added cmake build for osx #3468

Merged
merged 1 commit into from Jan 15, 2019
Merged

Conversation

@snikulov
Copy link
Member

@snikulov snikulov commented Jan 14, 2019

Added preliminary support to build curl on OSX image.

  • LDAP is disabled, due deprecation issues (need support for OpenDirectory Framework)

Fixes: #3467

@bagder
Copy link
Member

@bagder bagder commented Jan 14, 2019

Regarding OpenSSL I think it is mostly about pointing out the correct path. Point out the dir as /usr/local/opt/openssl. There's already an autotool build on osx doing that.

Loading

@snikulov
Copy link
Member Author

@snikulov snikulov commented Jan 14, 2019

Regarding OpenSSL I think it is mostly about pointing out the correct path. Point out the dir as /usr/local/opt/openssl. There's already an autotool build on osx doing that.

Yes. Will check it soon :)

Loading

@snikulov snikulov force-pushed the pr_travis_osx_cmake branch from c2c265c to a6d87b4 Jan 14, 2019
@snikulov snikulov requested a review from bagder Jan 14, 2019
@snikulov
Copy link
Member Author

@snikulov snikulov commented Jan 14, 2019

@bagder provide OpenSSL root folder for find_package(...).
Now will be built with OpenSSL from /usr/local/opt/openssl.

Loading

bagder
bagder approved these changes Jan 14, 2019
@snikulov snikulov merged commit 383fd9d into curl:master Jan 15, 2019
17 of 19 checks passed
Loading
@snikulov snikulov deleted the pr_travis_osx_cmake branch Jan 15, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Apr 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants