Revert "gh-36964: Add # sage_setup: distribution directives to all files, remove remaining # coding: utf-8"

This reverts commit 6ecb1d8, reversing
changes made to 8ea5214.

Author: roed314

bootstrap

@@ -232,7 +232,8 @@ save () {
 	src/environment-optional-3.[89].yml src/environment-optional-3.1[0-9].yml \
 	src/Pipfile \
 	src/pyproject.toml \
-	src/requirements.txt
+	src/requirements.txt \
+	src/setup.cfg
 
     # Update version
     echo "$NEWCONFVERSION" >$PKG/package-version.txt

build/bin/write-dockerfile.sh

@@ -236,7 +236,7 @@ $ADD Makefile VERSION.txt COPYING.txt condarc.yml README.md bootstrap bootstrap-
 $ADD config/config.rpath config/config.rpath
 $ADD src/doc/bootstrap src/doc/bootstrap
 $ADD src/bin src/bin
-$ADD src/Pipfile.m4 src/pyproject.toml.m4 src/requirements.txt.m4 src/VERSION.txt src/
+$ADD src/Pipfile.m4 src/pyproject.toml.m4 src/requirements.txt.m4 src/setup.cfg.m4 src/VERSION.txt src/
 $ADD m4 ./m4
 $ADD pkgs pkgs
 $ADD build ./build

build/pkgs/typing_extensions/version_requirements.txt

@@ -1,3 +1,3 @@
 # According to https://github.com/python/typing_extensions/blob/main/CHANGELOG.md,
 # version 4.4.0 adds another Python 3.11 typing backport
-typing_extensions >= 4.4.0; python_version<"3.11"
+typing_extensions >= 4.4.0; python_version<'3.11'

pkgs/sagemath-categories/known-test-failures.json

@@ -26,7 +26,7 @@
         "ntests": 28
     },
     "sage.categories.affine_weyl_groups": {
-        "ntests": 15
+        "ntests": 14
     },
     "sage.categories.algebra_ideals": {
         "failed": true,
@@ -148,7 +148,7 @@
     },
     "sage.categories.coxeter_groups": {
         "failed": true,
-        "ntests": 363
+        "ntests": 362
     },
     "sage.categories.cw_complexes": {
         "failed": true,
@@ -301,7 +301,7 @@
     },
     "sage.categories.finite_complex_reflection_groups": {
         "failed": true,
-        "ntests": 199
+        "ntests": 178
     },
     "sage.categories.finite_coxeter_groups": {
         "ntests": 6
@@ -493,6 +493,10 @@
         "failed": true,
         "ntests": 13
     },
+    "sage.categories.inner_product_spaces": {
+        "failed": true,
+        "ntests": 9
+    },
     "sage.categories.integral_domains": {
         "failed": true,
         "ntests": 22
@@ -779,7 +783,7 @@
     },
     "sage.cpython.debug": {
         "failed": true,
-        "ntests": 13
+        "ntests": 14
     },
     "sage.cpython.dict_del_by_value": {
         "failed": true,
@@ -822,6 +826,7 @@
         "ntests": 321
     },
     "sage.doctest.reporting": {
+        "failed": true,
         "ntests": 124
     },
     "sage.doctest.sources": {
@@ -1086,7 +1091,7 @@
     },
     "sage.misc.sageinspect": {
         "failed": true,
-        "ntests": 332
+        "ntests": 329
     },
     "sage.misc.superseded": {
         "failed": true,
@@ -1119,7 +1124,6 @@
         "ntests": 128
     },
     "sage.repl.configuration": {
-        "failed": true,
         "ntests": 22
     },
     "sage.repl.display.fancy_repr": {
@@ -1145,7 +1149,6 @@
         "ntests": 42
     },
     "sage.repl.inputhook": {
-        "failed": true,
         "ntests": 7
     },
     "sage.repl.interface_magic": {
@@ -1154,7 +1157,7 @@
     },
     "sage.repl.interpreter": {
         "failed": true,
-        "ntests": 114
+        "ntests": 118
     },
     "sage.repl.ipython_extension": {
         "failed": true,
@@ -1185,7 +1188,7 @@
     },
     "sage.repl.load": {
         "failed": true,
-        "ntests": 38
+        "ntests": 42
     },
     "sage.repl.preparse": {
         "failed": true,
@@ -1312,6 +1315,9 @@
     "sage.structure.global_options": {
         "ntests": 145
     },
+    "sage.structure.graphics_file": {
+        "ntests": 8
+    },
     "sage.structure.indexed_generators": {
         "failed": true,
         "ntests": 90

pkgs/sagemath-standard/setup.cfg

@@ -0,0 +1 @@
+../../src/setup.cfg

pkgs/sagemath-standard/setup.cfg.m4

@@ -0,0 +1 @@
+../../src/setup.cfg.m4

pkgs/sagemath-standard/setup.py

@@ -69,60 +69,25 @@
     from sage_setup.autogen import autogen_all
     autogen_all()
 
-    # TODO: This should be quiet by default
-    print("Discovering Python/Cython source code....")
-    t = time.time()
-    from sage.misc.package import is_package_installed_and_updated
-
-    distributions = [
-        '',
-        "sagemath-brial",
-        "sagemath-categories",
-        "sagemath-combinat",
-        "sagemath-eclib",
-        "sagemath-environment",
-        "sagemath-flint",
-        "sagemath-gap",
-        "sagemath-giac",
-        "sagemath-glpk",
-        "sagemath-graphs",
-        "sagemath-groups",
-        "sagemath-homfly",
-        "sagemath-lcalc",
-        "sagemath-libbraiding",
-        "sagemath-libecm",
-        "sagemath-linbox",
-        "sagemath-modules",
-        "sagemath-mpmath",
-        "sagemath-ntl",
-        "sagemath-objects",
-        "sagemath-pari",
-        "sagemath-plot",
-        "sagemath-polyhedra",
-        "sagemath-repl",
-        "sagemath-schemes",
-        "sagemath-singular",
-        "sagemath-symbolics",
-    ]
-
-    optional_packages_with_extensions = os.environ.get('SAGE_OPTIONAL_PACKAGES_WITH_EXTENSIONS', '').split(',')
-    distributions += ['sagemath-{}'.format(pkg)
-                      for pkg in optional_packages_with_extensions
-                      if is_package_installed_and_updated(pkg)]
-    log.warn('distributions = {0}'.format(distributions))
-    from sage_setup.find import find_python_sources
-    python_packages, python_modules, cython_modules = find_python_sources(
-        SAGE_SRC, ['sage'], distributions=distributions)
-
-    log.debug('python_packages = {0}'.format(python_packages))
-    log.debug('python_modules = {0}'.format(python_modules))
-    log.debug('cython_modules = {0}'.format(cython_modules))
-
-    print("Discovered Python/Cython sources, time: %.2f seconds." % (time.time() - t))
-else:
-    # sdist, egg_info, dist_info
-    python_packages = []
-    cython_modules = []
+# TODO: This should be quiet by default
+print("Discovering Python/Cython source code....")
+t = time.time()
+from sage.misc.package import is_package_installed_and_updated
+distributions = ['']
+optional_packages_with_extensions = os.environ.get('SAGE_OPTIONAL_PACKAGES_WITH_EXTENSIONS', '').split(',')
+distributions += ['sagemath-{}'.format(pkg)
+                  for pkg in optional_packages_with_extensions
+                  if is_package_installed_and_updated(pkg)]
+log.warn('distributions = {0}'.format(distributions))
+from sage_setup.find import find_python_sources
+python_packages, python_modules, cython_modules = find_python_sources(
+    SAGE_SRC, ['sage'], distributions=distributions)
+
+log.debug('python_packages = {0}'.format(python_packages))
+log.debug('python_modules = {0}'.format(python_modules))
+log.debug('cython_modules = {0}'.format(cython_modules))
+
+print("Discovered Python/Cython sources, time: %.2f seconds." % (time.time() - t))
 
 #########################################################
 ### Distutils

pkgs/sagemath-standard/tox.ini

@@ -17,7 +17,7 @@
 [tox]
 envlist =
     # Build and test without using the concrete dependencies specified by requirements.txt,
-    # using the dependencies declared in pyproject.toml (build-system requires, dependencies) only:
+    # using the dependencies declared in pyproject.toml and setup.cfg (install-requires) only:
     # Still use ONLY the wheels built and stored by the Sage distribution (no PyPI).
     #
     #   ./sage -sh -c '(cd pkgs/sagemath-standard && tox -v -v -v -e sagepython-sagewheels-nopypi-norequirements)'
@@ -45,7 +45,7 @@ envlist =
     #
     #   ./sage -sh -c '(cd pkgs/sagemath-standard && tox -v -v -v -e sagepython-sagewheels-pipenv-dist)'
     #
-    # Build using the dependencies declared in pyproject.toml (build-system requires, dependencies) only:
+    # Build using the dependencies declared in pyproject.toml and setup.cfg (install-requires) only.
     # Use the wheels built and stored by the Sage distribution,
     # and additionally allow packages from PyPI.
     #

src/Pipfile.m4

@@ -6,17 +6,17 @@ url = "https://pypi.org/simple"
 verify_ssl = true
 
 [dev-packages]
-## We do not list packages that are already declared as dependencies (install_requires)
-## in pyproject.toml
+## We do not list packages that are already declared as install_requires
+## in setup.cfg
 pycodestyle = "*"
 tox = "*"
 pytest = "*"
 rope = "*"
 six = "*"
 
 [packages]
-## We do not list packages that are already declared as dependencies (install_requires)
-## in pyproject.toml
+## We do not list packages that are already declared as install_requires
+## in setup.cfg
 
 [packages.e1839a8]
 path = "."

src/doc/en/developer/coding_basics.rst

@@ -193,8 +193,8 @@ included in one of the following places:
   This practice is deprecated, see :issue:`33037`.
 
 In all cases, the files must be listed (explicitly or via wildcards) in
-the section ``[tool.setuptools.package-data]`` of the file
-:sage_root:`SAGE_ROOT/pkgs/sagemath-standard/pyproject.toml.m4` (or the corresponding
+the section ``options.package_data`` of the file
+:sage_root:`pkgs/sagemath-standard/setup.cfg.m4` (or the corresponding
 file of another distribution).
 
 Large data files should not be added to the Sage source tree. Instead, it