-
-
Notifications
You must be signed in to change notification settings - Fork 412
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
[WIP] Use Conda.jl if Ipython/jupyter is not found #357
Conversation
@@ -2,4 +2,4 @@ julia 0.3 | |||
Nettle | |||
JSON 0.2 | |||
ZMQ 0.1 | |||
Compat 0.7 | |||
Compat 0.7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
I have updated the pull to a working condition. It do only work on master of Conda.jl. @Luthaf maybe you could tag a new version? While getting this to work I bumped into jupyter/notebook#448. I have added a workaround for the issue. |
@@ -3,3 +3,4 @@ Nettle | |||
JSON 0.2 | |||
ZMQ 0.1 | |||
Compat 0.7 | |||
Conda |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(please end the file with a newline)
|
7ba87a9
to
52c00d6
Compare
I have pushed a new version that fixes the workaround when IPython3 is used and the new lines. |
# Remove the commit that added this when https://github.com/jupyter/notebook/issues/448 is closed | ||
run(`$jupyter-kernelspec install --replace --user $juliakspec`) | ||
else | ||
run(`$jupyter-kernelspec install --replace --user $juliakspec`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this supposed to be $jupyter kernelspec
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it is.
jupyter/ipython is found.
jupyter/notebook#448 is closed and released.
@@ -78,4 +96,16 @@ copy_config("logo-32x32.png", juliakspec) | |||
copy_config("logo-64x64.png", juliakspec) | |||
|
|||
eprintln("Installing julia kernelspec $spec_name") | |||
run(`$jupyter kernelspec install --replace --user $juliakspec`) | |||
if basename(jupyter) == "jupyter" | |||
# Remove the commit that added this when https://github.com/jupyter/notebook/issues/448 is closed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just say # Just use "$jupyter kernelspec" when jupyter/notebook#448 is resolved and widely available.
I don't think I want to bother with a separate commit for this.
Looking good. Can you squash the commits? |
Oh nevermind, I guess we can leave it as two commits. |
[WIP] Use Conda.jl if Ipython/jupyter is not found
Yup, updating the README now. |
👏 🎆 |
This is the first working version of an integrated IPython setup on all platforms.
JUPYTER
environment variableconst jupyter = "..."
in a IJulia/deps/deps.jl file that is generated duringPkg.build
, and is included from IJulia.jl