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 case for Ubuntu 16.04 #19390

Merged
merged 1 commit into from
Nov 29, 2016
Merged

Conversation

TotalVerb
Copy link
Contributor

@TotalVerb TotalVerb commented Nov 23, 2016

This fixes #18693 for me.

fengyang@serval ~/julia> uname -a
Linux serval 4.4.0-47-generic #68-Ubuntu SMP Wed Oct 26 19:39:52 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

cc @tkelman

@tkelman tkelman added the libgit2 The libgit2 library or the LibGit2 stdlib module label Nov 23, 2016
@tkelman
Copy link
Contributor

tkelman commented Nov 23, 2016

Cool, thanks for testing. Only question I guess is if multiple of these are found, which should take priority?

@TotalVerb
Copy link
Contributor Author

TotalVerb commented Nov 23, 2016

Are there distros with multiple path, and distinct certificate files? That seems like a difficult decision to make, if there are. But probably for now we can ignore it.

@staticfloat
Copy link
Member

All distros I've seen where multiple of our paths match are symlinks to one concrete file. I've yet to see a system that has multiple certificate stores with different contents in them, and if one does exist, I'm not sure we can possibly know which is the best, unless we wanted to somehow test that the certificates work. (I don't think we want to do that)

@TotalVerb
Copy link
Contributor Author

Is this good to merge then?

@tkelman
Copy link
Contributor

tkelman commented Nov 25, 2016

We should verify that in an actual example, are these symlinks to the same file when multiple of them exist? If they're symlinks, does this copy them correctly (should it be dereferencing the symlink)?

@davidanthoff
Copy link
Contributor

I just tested this on Bash on Ubuntu on Windows (or whatever the name is), and this fixes the bug there as well. This is using Ubuntu 14.04.

@TotalVerb
Copy link
Contributor Author

Bump.

@staticfloat
Copy link
Member

We should verify that in an actual example, are these symlinks to the same file when multiple of them exist? If they're symlinks, does this copy them correctly (should it be dereferencing the symlink)?

The only distribution I've been able to find that has both of these facts true is CentOS, which has OPENSSLDIR set to /etc/pki/tls, and both $(OPENSSL_DIR)/cert.pem and $(OPENSSL_DIR)/ca-bundle.crt available. (Note that ca-bundle.crt is not looked for by Julia, but is a second valid certificate store name).

Both of those certificate stores are symlinks to a third file, located at /etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem. Julia correctly dereferences the symlink and copies the file contents into a normal file. Indeed, this is the same even on the CentOS 5.11 image that our binary builds are produced on.

@tkelman
Copy link
Contributor

tkelman commented Nov 29, 2016

I've also seen Ubuntu installs that had multiple of these names present. Should look into whether they're all equivalent.

@staticfloat
Copy link
Member

staticfloat commented Nov 29, 2016 via email

@tkelman tkelman merged commit 4f18204 into JuliaLang:master Nov 29, 2016
@staticfloat
Copy link
Member

Thanks @TotalVerb for the contribution and @davidanthoff for the testing on Windows!

fcard pushed a commit to fcard/julia that referenced this pull request Feb 28, 2017
@ziotom78 ziotom78 mentioned this pull request Jan 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libgit2 The libgit2 library or the LibGit2 stdlib module system:linux Affects only Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"error initializing LibGit2 module" when CA certificates not found
5 participants