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/ipython: Version bump + possible fix for test phase #1571

Closed
wants to merge 2 commits into from

Conversation

marbre
Copy link
Contributor

@marbre marbre commented May 31, 2016

@jlec @SoapGentoo

I hopefully fixed the call of run_tests() in 4.1.2 which was accidentally broken due to the EAPI bump. However, tests currently fail on my machine with

 * Call stack:
 *     ebuild.sh, line  133:  Called src_test
 *   environment, line 3837:  Called distutils-r1_src_test
 *   environment, line 1074:  Called _distutils-r1_run_foreach_impl 'python_test'
 *   environment, line  409:  Called python_foreach_impl 'distutils-r1_run_phase' 'python_test'
 *   environment, line 3330:  Called multibuild_foreach_variant '_python_multibuild_wrapper' 'distutils-r1_run_phase' 'python_test'
 *   environment, line 2421:  Called _multibuild_run '_python_multibuild_wrapper' 'distutils-r1_run_phase' 'python_test'
 *   environment, line 2419:  Called _python_multibuild_wrapper 'distutils-r1_run_phase' 'python_test'
 *   environment, line  697:  Called distutils-r1_run_phase 'python_test'
 *   environment, line 1007:  Called python_test
 *   environment, line 3716:  Called virtx 'run_tests'
 *   environment, line 4429:  Called die
 * The specific snippet of code:
 *       XHOST=$(type -p xhost) || die;

Anyway, this is the case for all ipython ebuilds currently in the tree and might be related to my machine. The EAPI=5 ebuils fail with a similar message, also additionally virtalmake is called before vitx.

Furthermore the version bump adds a new dependency (prompt_tool) to the ipython ebuild and adds a more specific dependency on traitlets.

Package-Manager: portage-2.2.28
@monsieurp monsieurp added version bump assigned PR successfully assigned to the package maintainer(s). labels Jun 1, 2016
@gentoo-bot gentoo-bot closed this in e8b6ebd Jun 4, 2016
gentoo-bot pushed a commit that referenced this pull request Jun 4, 2016
Package-Manager: portage-2.2.28
Closes: #1571

Signed-off-by: David Seifert <soap@gentoo.org>
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