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 PHP Stack #444

Closed
wants to merge 3 commits into from
Closed

Add PHP Stack #444

wants to merge 3 commits into from

Conversation

to-masz
Copy link

@to-masz to-masz commented Aug 22, 2017

PHP stack based on official Base Jupyter Notebook Stack

It comes with PHP 7.0 and Jupyter-PHP kernel


# clean
RUN apt-get clean && \
rm -rf /var/lib/apt/lists/*
Copy link
Member

Choose a reason for hiding this comment

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

clean has to always be in the same RUN line as install, otherwise it doesn't help.

wget https://download.opensuse.org/repositories/network:/messaging:/zeromq:/release-stable/Debian_9.0/Release.key -O- | sudo apt-key add && \
apt-get update && \
apt-get install -y --no-install-recommends \
libzmq3-dev
Copy link
Member

Choose a reason for hiding this comment

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

It would be best if all of the apt commands could be a single RUN, including clean.

@parente
Copy link
Member

parente commented Aug 22, 2017

@to-masz thanks for the contribution. Can you help out maintaining here?

@to-masz
Copy link
Author

to-masz commented Aug 24, 2017

@parente sure

@minrk please review the changes and verify if they fixes reported issue

@minrk
Copy link
Member

minrk commented Aug 26, 2017

Looks okay, thanks!

@parente what do we need to do with respect to build hooks, etc. to get a new stack building?

@parente
Copy link
Member

parente commented Aug 26, 2017

@minrk If you want to sanity-check the steps I wrote for setting up Docker Cloud, they're in https://github.com/jupyter/docker-stacks#maintainer-workflow under the "When there's a new stack definition, do the following before merging the PR with the new stack:" section. If you're hacking on other things, I can get to them later.


## What it Gives You

* Minimally-functional Jupyter Notebook 5.0.x (e.g., no pandoc for document conversion)
Copy link
Member

Choose a reason for hiding this comment

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

The Dockerfile is based on jupyter/minimal-notebook which does have pandoc. See README here: https://github.com/jupyter/docker-stacks/tree/master/minimal-notebook

Did you mean to base it on base-notebook?

USER root

RUN echo "deb http://download.opensuse.org/repositories/network:/messaging:/zeromq:/release-stable/Debian_9.0/ ./" >> /etc/apt/sources.list && \
wget https://download.opensuse.org/repositories/network:/messaging:/zeromq:/release-stable/Debian_9.0/Release.key -O- | sudo apt-key add
Copy link
Member

Choose a reason for hiding this comment

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

Does it make sense to be pulling packages from opensuse for debian when the images are based on ubuntu?

RUN cd /tmp && \
wget https://litipk.github.io/Jupyter-PHP-Installer/dist/jupyter-php-installer.phar && \
chmod +x jupyter-php-installer.phar && \
./jupyter-php-installer.phar install -vv
Copy link
Member

Choose a reason for hiding this comment

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

Out of curiosity, to what directory does this install the kernel and the kernel.json kernelspec?

@parente parente added the status:Need Info We believe we need more information about an issue from the reporting user to help, debug, fix label Sep 9, 2017
@parente
Copy link
Member

parente commented Jun 14, 2018

Hi @to-masz. We'd like to start encouraging the growth of community-maintained stacks in other repositories as an alternative to continually expanding the set and scope of images maintained here. I think this image would make a fine addition to the ecosystem we're trying to cultivate, and would love it if you were to be one of the first to follow the new community stacks documentation to set up a project.

If you do decide to give it a shot, please feel free to submit a PR updating the documentation site with a link to the git repository. (The community stacks documentation includes this suggestion along with steps to do it.)

@parente parente added tag:Community Stack We recommend this contribution become a community stack. and removed status:Need Info We believe we need more information about an issue from the reporting user to help, debug, fix labels Jun 14, 2018
@parente
Copy link
Member

parente commented Sep 3, 2018

I'm going to close this PR as inactive. If someone picks up the torch and creates a community maintained PHP image, please submit a PR to linking to it from the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag:Community Stack We recommend this contribution become a community stack.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants