Extract cython options to new module

Author: tobiasdiez

src/sage_setup/command/sage_build_cython.py

@@ -17,6 +17,7 @@
 from sage_setup.util import stable_uniq
 from sage_setup.find import find_extra_files
 from sage_setup.library_order import library_order
+from sage_setup.cython_options import compiler_directives, compile_time_env_variables
 
 from sage.env import (cython_aliases, sage_include_directories)
 
@@ -149,23 +150,8 @@ def finalize_options(self):
                 "Cython must be installed and importable in order to run "
                 "the cythonize command")
 
-        # Cython compiler directives
-        self.cython_directives = dict(
-            auto_pickle=False,
-            autotestdict=False,
-            cdivision=True,
-            embedsignature=True,
-            fast_getattr=True,
-            language_level="3str",
-            preliminary_late_includes_cy28=True,
-            profile=self.profile,
-        )
-        self.compile_time_env = dict(
-            PY_PLATFORM=sys.platform,
-            # The following two constants are here only for backwards compatibility of user packages
-            PY_VERSION_HEX=sys.hexversion,
-            PY_MAJOR_VERSION=sys.version_info[0]
-        )
+        self.cython_directives = compiler_directives(self.profile)
+        self.compile_time_env = compile_time_env_variables()
 
         # We check the Cython version and some relevant configuration
         # options from the earlier build to see if we need to force a

src/sage_setup/cython_options.py

@@ -0,0 +1,32 @@
+import sys
+
+def compiler_directives(profile: bool):
+    """
+    Returns a list of Cython directives used for compilation.
+    """
+    return dict(
+        # Do not create __test__ dictionary automatically from docstrings
+        autotestdict=False,
+        # Do not check for division by 0 (this is about 35% quicker than with check)
+        cdivision=True,
+        # Embed a textual copy of the call signature in the docstring (to support tools like IPython)
+        embedsignature=True,
+        fast_getattr=True,
+        # Use Python 3 (including source code semantics) for module compilation
+        language_level="3str",
+        # Enable support for late includes (make declarations in Cython code available to C include files)
+        preliminary_late_includes_cy28=True,
+        # Add hooks for Python profilers into the compiled C code
+        profile=profile,
+    )
+
+def compile_time_env_variables():
+    """
+    Returns a list of environmental variables used for compilation.
+    """
+    return dict(
+        PY_PLATFORM=sys.platform,
+        # The following two constants are here only for backwards compatibility of user packages
+        PY_VERSION_HEX=sys.hexversion,
+        PY_MAJOR_VERSION=sys.version_info[0]
+    )