Skip to content
This repository was archived by the owner on Feb 1, 2023. It is now read-only.

Commit 5a76fdc

Browse files
author
Matthias Koeppe
committed
build/pkgs/snappy: Update dependencies, requirements.txt, add comments
1 parent 46f7b0e commit 5a76fdc

File tree

2 files changed

+15
-4
lines changed

2 files changed

+15
-4
lines changed

build/pkgs/snappy/dependencies

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,10 @@
1-
$(PYTHON) | $(PYTHON_TOOLCHAIN)
1+
$(PYTHON) decorator future ipython cypari | $(PYTHON_TOOLCHAIN) sagelib
22

33
----------
44
All lines of this file are ignored except the first.
5+
6+
The dependency cypari above is actually cypari2.
7+
8+
An installed sagelib is needed when snappy is installed from source (instead of a wheel)
9+
because snappy's setup.py tests its presence to adjust dependencies.
10+
https://trac.sagemath.org/ticket/31180

build/pkgs/snappy/requirements.txt

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1-
# The --no-deps flag is just there to prevent it from pulling in cypari (!= cypari2).
2-
# In fact, just sage -pip install snappy does the trick but installs the (unused even by SnapPy in this context) cypari module.
3-
--no-deps snappy spherogram plink FXrays snappy_manifolds
1+
# Note: As of 2021-01, snappy will pull in cypari (!= cypari2) as a dependency
2+
# if installed as a wheel but will actually use Sage's cypari2.
3+
# cypari contains a statically linked copy of pari and other libraries
4+
# and will remain completely unused (wastes 30M). Snappy is about 165M.
5+
# See https://trac.sagemath.org/ticket/31180
6+
snappy
7+
# An optional database (110M uncompressed)
8+
snappy_15_knots

0 commit comments

Comments
 (0)