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

update gnutls packages for Debian #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Quantisan
Copy link

Closes #43

Tested on Debian Jessie and working.

@nalimilan
Copy link

Ah, sorry, turns out I was wrong about the changes I suggested. WinRPM.RPM supports this syntax, but AptGet and Yum do not. Looks like you need to do something like this:

provides(AptGet, @compat Dict("libgnutls28"=>gnutls, "libgnutls-deb0-28"=>gnutls))

I'll discuss this issue in BinDeps at JuliaPackaging/BinDeps.jl#147.

@Quantisan
Copy link
Author

It's weird that my install worked if this isn't supported yet. Perhaps I'm trying it wrong?

I started a fresh Docker container using julia:0.3 image, which uses Debian Jessie. Then I run julia and do a Pkg.clone(<link to my forked github>). It installed fine for some reason.

@nalimilan
Copy link

Well, no idea, but it fails the Travis checks, and it didn't work locally here on 0.4... Maybe gnutls was already on the image?

@staticfloat
Copy link
Contributor

The proper way to do this is like this:

provides(AptGet,"libgnutls28",gnutls,validate = pkgmanager_validate)
provides(AptGet,"libgnutls-deb0-28",gnutls,validate = pkgmanager_validate)

Note that you would also need to add in a new library name alias, I submitted a PR to the same effect just now, I didn't see this one before. Feel free to incorporate my changes into this PR; if you do so, I'll close mine.

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