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

Bump libgit2, document libssl-dev build dep and fail earlier when missing #13448

Merged
merged 3 commits into from
Oct 6, 2015

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented Oct 5, 2015

closes #13389

The REQUIRE_OPENSSL flag isn't used yet but I've proposed a PR to libgit2 upstream so it wouldn't be needed as a patch in future versions.

@wildart
Copy link
Member

wildart commented Oct 5, 2015

I guess openssl requirements is a bit strict for libgit2 upstream, but for us it's a necessity. So, merge it.

@tkelman
Copy link
Contributor Author

tkelman commented Oct 5, 2015

My patch for upstream is a little more complicated than the one here, adding a new option that's off by default so you could flag if you want the dependency to be mandatory. Let's see whether they approve of the idea.

@yuyichao
Copy link
Contributor

yuyichao commented Oct 5, 2015

Would be nice if there's a CMAKE_DISABLE_FIND_PACKAGE_<PackageName> equivalence to REQUIRE.

@tkelman
Copy link
Contributor Author

tkelman commented Oct 5, 2015

Maybe we should have used that for the iconv issues on mac...

tkelman added a commit that referenced this pull request Oct 6, 2015
Bump libgit2, document libssl-dev build dep and fail earlier when missing
@tkelman tkelman merged commit a3d04ab into master Oct 6, 2015
@tkelman tkelman deleted the tk/require_openssl branch October 6, 2015 05:46
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.

Document dependency on libssl when compiling Julia from source
3 participants