Skip to content

Convert Dockerfile to be based on Debian Jessie#109

Merged
danlamanna merged 3 commits into
masterfrom
debian-dockerfile
Mar 29, 2017
Merged

Convert Dockerfile to be based on Debian Jessie#109
danlamanna merged 3 commits into
masterfrom
debian-dockerfile

Conversation

@danlamanna

Copy link
Copy Markdown
Contributor

No description provided.

@danlamanna danlamanna requested a review from dorukozturk March 27, 2017 17:08
@danlamanna

Copy link
Copy Markdown
Contributor Author

I should note the relevant issue within Jupyter that requires us to run the notebook through a shell script: ipython/ipykernel#233

@aashish24

Copy link
Copy Markdown
Contributor

thanks @danlamanna I will give it a try soon.

@dorukozturk

Copy link
Copy Markdown
Contributor

@danlamanna This worked for me

@aashish24 aashish24 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me on Mac 👍

@aashish24

Copy link
Copy Markdown
Contributor

@danlamanna I did not see this message as mentioned in the document (https://github.com/OpenGeoscience/geonotebook/tree/master/devops/docker) but it is possible that I may have missed something.

[C 20:30:42.834 NotebookApp]

Copy/paste this URL into your browser when you connect for the first time,
to login with a token:
    http://localhost:8888/?token=4a4fe605780711fb0aea99b55836142c95af157277ac25d5

@danlamanna

Copy link
Copy Markdown
Contributor Author

That was supposed to be disabled for Docker builds, I'll update the docs.

@aashish24

Copy link
Copy Markdown
Contributor

That was supposed to be disabled for Docker builds, I'll update the docs.

Great, thanks.

@aashish24

Copy link
Copy Markdown
Contributor

LGTM +2 thanks for updating the docs.

@danlamanna danlamanna merged commit 187a4e2 into master Mar 29, 2017
@danlamanna danlamanna deleted the debian-dockerfile branch March 29, 2017 14:13
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