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

install Julia packages in /opt/julia #386

Merged
merged 1 commit into from
May 27, 2017
Merged

Conversation

minrk
Copy link
Member

@minrk minrk commented May 17, 2017

This gets close to #331, but we're not quite there. I believe the only files left in $HOME are .curlrc and .ipython/profile_default/startup/mplimporthook

@parente do you recall what the curlrc config solves and/or if it can be put in /etc/ somewhere?

@parente
Copy link
Member

parente commented May 18, 2017

The .curlrc fixes a problem with using curl to access sites over HTTPS:

echo "cacert=/etc/ssl/certs/ca-certificates.crt" > /home/$NB_USER/.curlrc
. Without it, every certificate was rejected as untrusted long ago. Maybe newer images don't have this problem or maybe there's a better way to configure it. https://ec.haxx.se/cmdline-configfile.html seems to suggest curl only looks in the home directory on linux.

@jakirkham
Copy link
Member

Is this just the system curl? We built curl at conda-forge as well, which uses certificates that we pulled from the certifi package. It generally works ok.

ref: http://github.com/conda-forge/curl-feedstock
ref: http://github.com/conda-forge/ca-certificates-feedstock

@parente
Copy link
Member

parente commented May 18, 2017

It does indeed predate most of conda forge. It's worth trying to switch to a curl package installed in the root conda env so that it's available on the default path.

@minrk
Copy link
Member Author

minrk commented May 20, 2017

I just ran in a base-notebook container:

rm ~/.bashrc
apt update && apt install curl
curl https://github.com

and it works fine. Does that mean we don't need this file anymore?

@jakirkham
Copy link
Member

Maybe it is worth digging through git blame to see who added the file and why. Just in case there is something we are missing. Then again I don't know much about it.

@parente
Copy link
Member

parente commented May 20, 2017

I'm pretty sure I added the .curlrc. I'll go poke through the history.

@parente
Copy link
Member

parente commented May 20, 2017

Nope, it was in be4ca70 for issue #153, cert error when Julia downloaded the QuantEcon package.

@parente
Copy link
Member

parente commented May 27, 2017

Let's get this in. If curl breaks, we can always build again with a fix.

@parente parente merged commit 174fc49 into jupyter:master May 27, 2017
@minrk minrk deleted the system-julia branch May 28, 2017 01:22
@jakirkham
Copy link
Member

Should add we have julia in conda-forge now.

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