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

dev-python/jupyter_core: Drop old + Version bump #4042

Closed
wants to merge 2 commits into from

Conversation

marbre
Copy link
Contributor

@marbre marbre commented Feb 20, 2017

@SoapGentoo Version bump to 4.3.0. Revised doc building/installation and python2_7 deps.

Package-Manager: portage-2.3.3
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull Request assignment

Areas affected: ebuilds
Packages affected: dev-python/jupyter_core

dev-python/jupyter_core: @gentoo/sci

@gentoo-repo-qa-bot gentoo-repo-qa-bot added the assigned PR successfully assigned to the package maintainer(s). label Feb 20, 2017
Copy link
Contributor

@jlec jlec left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -50,6 +53,5 @@ python_test() {
}

python_install_all() {
use doc && HTML_DOCS=( docs/_build/html/. )
distutils-r1_python_install_all
}
Copy link
Member

Choose a reason for hiding this comment

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

chuck this whole phase function, just calling the default is what happens anyway

Package-Manager: portage-2.3.3
@marbre
Copy link
Contributor Author

marbre commented Feb 20, 2017

@SoapGentoo Dropped python_install_all().

gentoo-bot pushed a commit that referenced this pull request Feb 21, 2017
Package-Manager: portage-2.3.3
Closes: #4042
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s).
Projects
None yet
4 participants