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

Race condition: Sage library now depends on pip #21403

Closed
nexttime mannequin opened this issue Sep 2, 2016 · 6 comments
Closed

Race condition: Sage library now depends on pip #21403

nexttime mannequin opened this issue Sep 2, 2016 · 6 comments

Comments

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 2, 2016

(We might in the future allow to use a system-wide pip though, but that requires changing configure[.ac] as well.)

[sagelib-7.4.beta2] Building interpreters for fast_callable
[sagelib-7.4.beta2] python -u setup.py install
[sagelib-7.4.beta2] ************************************************************************
[sagelib-7.4.beta2] Traceback (most recent call last):
[sagelib-7.4.beta2]   File "setup.py", line 47, in <module>
[sagelib-7.4.beta2]     from module_list import ext_modules, library_order, aliases
[sagelib-7.4.beta2]   File "/home/agag/sbrandhorst/sage/sage/src/module_list.py", line 150, in <module>
[sagelib-7.4.beta2]     from sage_setup.optional_extension import OptionalExtension
[sagelib-7.4.beta2]   File "/home/agag/sbrandhorst/sage/sage/src/sage_setup/optional_extension.py", line 24, in <module>
[sagelib-7.4.beta2]     all_packages = list_packages(local=True)
[sagelib-7.4.beta2]   File "/home/agag/sbrandhorst/sage/sage/src/sage/misc/package.py", line 211, in list_packages
[sagelib-7.4.beta2]     installed = installed_packages()
[sagelib-7.4.beta2]   File "/home/agag/sbrandhorst/sage/sage/src/sage/misc/package.py", line 293, in installed_packages
[sagelib-7.4.beta2]     installed.update(pip_installed_packages())
[sagelib-7.4.beta2]   File "/home/agag/sbrandhorst/sage/sage/src/sage/misc/package.py", line 147, in pip_installed_packages
[sagelib-7.4.beta2]     proc = subprocess.Popen(["pip", "list"], stdout=subprocess.PIPE)
[sagelib-7.4.beta2]   File "/home/agag/sbrandhorst/sage/sage/local/lib/python/subprocess.py", line 710, in __init__
[sagelib-7.4.beta2]     errread, errwrite)
[sagelib-7.4.beta2]   File "/home/agag/sbrandhorst/sage/sage/local/lib/python/subprocess.py", line 1335, in _execute_child
[sagelib-7.4.beta2]     raise child_exception
[sagelib-7.4.beta2] OSError: [Errno 2] No such file or directory
[sagelib-7.4.beta2] ************************************************************************
[sagelib-7.4.beta2] Error building the Sage library
[sagelib-7.4.beta2] ************************************************************************
[sagelib-7.4.beta2] Please email sage-devel (http://groups.google.com/group/sage-devel)
[sagelib-7.4.beta2] explaining the problem and including the relevant part of the log file
[sagelib-7.4.beta2]   /home/agag/sbrandhorst/sage/sage/logs/pkgs/sagelib-7.4.beta2.log
[sagelib-7.4.beta2] Describe your computer, operating system, etc.
[sagelib-7.4.beta2] ************************************************************************
[sagelib-7.4.beta2] make[3]: *** [sage] Error 1
[sagelib-7.4.beta2] make[3]: Leaving directory `/home/agag/sbrandhorst/sage/sage/src'
[sagelib-7.4.beta2]
[sagelib-7.4.beta2] real    0m27.900s
[sagelib-7.4.beta2] user    0m17.900s
[sagelib-7.4.beta2] sys    0m4.588s
make[2]: *** [sagelib] Error 2
make[2]: Leaving directory `/home/agag/sbrandhorst/sage/sage/build/make'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/home/agag/sbrandhorst/sage/sage/build/make'

real    303m54.652s
user    113m57.245s
sys    23m9.735s
***************************************************************
Error building Sage.

Cf. my post to sage-release, as well as the original report on sage-devel (of September 2nd).

Component: build

Keywords: build error, OSError, Popen, optional_extension, list_packages

Author: Leif Leonhardy

Branch/Commit: cf54771

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/21403

@nexttime nexttime mannequin added this to the sage-7.4 milestone Sep 2, 2016
@nexttime nexttime mannequin added c: build labels Sep 2, 2016
@nexttime
Copy link
Mannequin Author

nexttime mannequin commented Sep 2, 2016

New commits:

cf54771Trac #21403: Race condition: Sage library now depends on pip -- add dependency

@nexttime
Copy link
Mannequin Author

nexttime mannequin commented Sep 2, 2016

Branch: u/leif/sagelib_depends_on_pip

@nexttime
Copy link
Mannequin Author

nexttime mannequin commented Sep 2, 2016

Commit: cf54771

@nexttime
Copy link
Mannequin Author

nexttime mannequin commented Sep 2, 2016

Author: Leif Leonhardy

@nexttime nexttime mannequin added the s: needs review label Sep 2, 2016
@vbraun
Copy link
Member

vbraun commented Sep 2, 2016

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Sep 4, 2016

Changed branch from u/leif/sagelib_depends_on_pip to cf54771

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant