Update to new conway-polynomials python package

build/pkgs/conway_polynomials/SPKG.rst

@@ -1,12 +1,26 @@
-conway_polynomials: Tables of Conway polynomials over finite fields
-===================================================================
+conway_polynomials: Python interface to Frank Lübeck's Conway polynomial database
+=================================================================================
 
 Description
 -----------
 
-Frank Lübeck's tables of Conway polynomials over finite fields.
+This python module evolved from the old SageMath *conway_polynomials*
+package once hosted at,
 
-Upstream contact
+  http://files.sagemath.org/spkg/upstream/conway_polynomials/
+
+It's still maintained by Sage developers, but having a pip-installable
+interface to the data will make it easier to install SageMath via pip
+or another package manager.
+
+
+License
+-------
+
+GPL version 3 or later
+
+
+Upstream Contact
 ----------------
 
-http://www.math.rwth-aachen.de/~Frank.Luebeck/data/ConwayPol/
+https://github.com/sagemath/conway-polynomials

build/pkgs/conway_polynomials/checksums.ini

@@ -1,4 +1,5 @@
-tarball=conway_polynomials-VERSION.tar.bz2
-sha1=d4d89bda60ac54e73121f84635e774766e19a8b6
-md5=a2725ba21f44554196781424d957f68a
-cksum=1387933493
+tarball=conway-polynomials-VERSION.tar.gz
+sha1=a822baa014c68ab44f654d8a88b2ea81436a26ef
+md5=779cfaad6fc4413fa5b4d5a17e5e9ceb
+cksum=634397699
+upstream_url=https://files.pythonhosted.org/packages/source/c/conway-polynomials/conway-polynomials-VERSION.tar.gz

build/pkgs/conway_polynomials/dependencies

@@ -1,4 +1,4 @@
-| $(PYTHON)
+| $(PYTHON_TOOLCHAIN) $(PYTHON)
 
 ----------
 All lines of this file are ignored except the first.

build/pkgs/conway_polynomials/distros/gentoo.txt

@@ -0,0 +1 @@
+dev-python/conway-polynomials

build/pkgs/conway_polynomials/install-requires.txt

@@ -0,0 +1 @@
+conway-polynomials >=0.7

build/pkgs/conway_polynomials/package-version.txt

@@ -1 +1 @@
-0.5
+0.7

build/pkgs/conway_polynomials/spkg-configure.m4

@@ -0,0 +1,3 @@
+SAGE_SPKG_CONFIGURE([conway_polynomials], [
+  SAGE_PYTHON_PACKAGE_CHECK([conway_polynomials])
+])

build/pkgs/conway_polynomials/spkg-install.in

@@ -1 +1 @@
-exec python3 spkg-install.py
+cd src && sdh_pip_install .

src/sage/databases/all.py

@@ -23,7 +23,7 @@
 EXAMPLES::
 
     sage: ConwayPolynomials()
-    Frank Luebeck's database of Conway polynomials
+    Frank Lübeck's database of Conway polynomials
 
     sage: CremonaDatabase()
     Cremona's database of elliptic curves with conductor...

src/sage/databases/conway.py

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 r"""
-Frank Luebeck's tables of Conway polynomials over finite fields
+Frank Lübeck's tables of Conway polynomials over finite fields
 """
 # ****************************************************************************
 #
@@ -15,13 +15,9 @@
 #                  https://www.gnu.org/licenses/
 # ****************************************************************************
 from collections.abc import Mapping
-import pickle
 
 from sage.features.databases import DatabaseConwayPolynomials
 
-_conwaydict = None
-
-
 class DictInMapping(Mapping):
     def __init__(self, dict):
         """
@@ -95,14 +91,10 @@ def __init__(self):
 
             sage: c = ConwayPolynomials()
             sage: c
-            Frank Luebeck's database of Conway polynomials
+            Frank Lübeck's database of Conway polynomials
         """
-        global _conwaydict
-        if _conwaydict is None:
-            _CONWAYDATA = DatabaseConwayPolynomials().absolute_filename()
-            with open(_CONWAYDATA, 'rb') as f:
-                _conwaydict = pickle.load(f)
-        self._store = _conwaydict
+        import conway_polynomials
+        self._store = conway_polynomials.database()
 
     def __repr__(self):
         """
@@ -112,9 +104,9 @@ def __repr__(self):
 
             sage: c = ConwayPolynomials()
             sage: c.__repr__()
-            "Frank Luebeck's database of Conway polynomials"
+            "Frank Lübeck's database of Conway polynomials"
         """
-        return "Frank Luebeck's database of Conway polynomials"
+        return "Frank Lübeck's database of Conway polynomials"
 
     def __getitem__(self, key):
         """
@@ -152,7 +144,7 @@ def __len__(self):
 
             sage: c = ConwayPolynomials()
             sage: len(c)
-            35352
+            35357
         """
         try:
             return self._len

src/sage/env.py

@@ -195,7 +195,6 @@ def var(key: str, *fallbacks: Optional[str], force: bool = False) -> Optional[st
 SAGE_PKG_CONFIG_PATH = var("SAGE_PKG_CONFIG_PATH")
 
 # installation directories for various packages
-CONWAY_POLYNOMIALS_DATA_DIR = var("CONWAY_POLYNOMIALS_DATA_DIR", join(SAGE_SHARE, "conway_polynomials"))
 GRAPHS_DATA_DIR = var("GRAPHS_DATA_DIR", join(SAGE_SHARE, "graphs"))
 ELLCURVE_DATA_DIR = var("ELLCURVE_DATA_DIR", join(SAGE_SHARE, "ellcurves"))
 POLYTOPE_DATA_DIR = var("POLYTOPE_DATA_DIR", join(SAGE_SHARE, "reflexive_polytopes"))

src/sage/features/__init__.py

@@ -254,7 +254,7 @@ def __repr__(self):
 
             sage: from sage.features.databases import DatabaseConwayPolynomials
             sage: DatabaseConwayPolynomials()  # indirect doctest
-            Feature('conway_polynomials': Frank Luebeck's database of Conway polynomials)
+            Feature('conway_polynomials': Frank Lübeck's database of Conway polynomials)
         """
         description = f'{self.name!r}: {self.description}' if self.description else f'{self.name!r}'
         return f'Feature({description})'

src/sage/features/databases.py

@@ -19,15 +19,15 @@
 
 from . import StaticFile, PythonModule
 from sage.env import (
-    CONWAY_POLYNOMIALS_DATA_DIR,
     CREMONA_MINI_DATA_DIR, CREMONA_LARGE_DATA_DIR,
     POLYTOPE_DATA_DIR)
 
 
-class DatabaseConwayPolynomials(StaticFile):
+class DatabaseConwayPolynomials(PythonModule):
     r"""
-    A :class:`~sage.features.Feature` which describes the presence of :ref:`Frank Luebeck's
-    database of Conway polynomials <spkg_conway_polynomials>`.
+    A :class:`~sage.features.Feature` which describes the presence of
+    :ref:`Frank Lübeck's database of Conway polynomials
+    <spkg_conway_polynomials>`.
 
     EXAMPLES::
 
@@ -44,16 +44,13 @@ def __init__(self):
             sage: isinstance(DatabaseConwayPolynomials(), DatabaseConwayPolynomials)
             True
         """
-        if CONWAY_POLYNOMIALS_DATA_DIR:
-            search_path = [CONWAY_POLYNOMIALS_DATA_DIR]
-        else:
-            search_path = []
-        StaticFile.__init__(self, "conway_polynomials",
-                            filename='conway_polynomials.p',
-                            search_path=search_path,
-                            spkg='conway_polynomials',
-                            description="Frank Luebeck's database of Conway polynomials",
-                            type='standard')
+        PythonModule.__init__(
+            self,
+            "conway_polynomials",
+            spkg='conway_polynomials',
+            description="Frank Lübeck's database of Conway polynomials",
+            type="standard"
+        )
 
 
 CREMONA_DATA_DIRS = set([CREMONA_MINI_DATA_DIR, CREMONA_LARGE_DATA_DIR])