Move Extension options from src/module_list.py to "distutils:" directives in the individual files (part 6: last)
Closed this issue · 18 comments
mkoeppe commented
Follow-up from #29706.
It takes care of all remaining extensions.
Depends on #29706
CC: @kliem
Component: refactoring
Author: Matthias Koeppe
Branch: 802356a
Reviewer: Jonathan Kliem
Issue created by migration from https://trac.sagemath.org/ticket/29791
mkoeppe commented
mkoeppe commented
Author: Matthias Koeppe
mkoeppe commented
Last 10 new commits:
70901ea | src/sage/env.py (cython_aliases): Update doctest |
eed920e | src/sage/tests: Move Extension options from src/module_list.py to distutils directives |
0d25d1d | src/sage/structure: Move Extension options from src/module_list.py to distutils directives |
808f46a | src/sage/stats: Move Extension options from src/module_list.py to distutils directives |
10f7542 | src/sage/schemes: Move Extension options from src/module_list.py to distutils directives |
5867c05 | src/sage/modular/pollack_stevens/dist.pyx: Add missing distutils directives |
c536daa | Remove self-listing in distutils sources directive |
9dc7022 | Merge branch 't/29706/move_extension_options_from_src_module_list_py_to__distutils___directives_in_the_individual_files' into t/29791/move_extension_options_from_src_module_list_py_to__distutils___directives_in_the_individual_files__part_6__last_ |
96b99c8 | src/module_list.py: Move options for Extensions in sage.matrix to distutils directives |
2355836 | src/module_list.py: Move remaining options for Extensions in sage.libs, sage.rings to distutils directives |
mkoeppe commented
Description changed:
---
+++
@@ -1 +1,5 @@
-Follow-up from #29706
+Follow-up from #29706.
+
+It takes care of all remaining extensions.
+
+sagetrac-git commented
sagetrac-git commented
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
f78b06d | src/module_list.py: Move options for Extensions in sage.libs to distutils directives |
1b0e29d | src/module_list.py: Move options for Extensions in sage.matrix to distutils directives |
6421e2c | src/module_list.py: Move remaining options for Extensions in sage.libs, sage.rings to distutils directives |
sagetrac-git commented
sagetrac-git commented
Branch pushed to git repo; I updated commit sha1. New commits:
b1b3787 | sage.env.cython_aliases: Fix for systems without zlib pc |
802356a | Merge branch 't/29706/move_extension_options_from_src_module_list_py_to__distutils___directives_in_the_individual_files' into t/29791/move_extension_options_from_src_module_list_py_to__distutils___directives_in_the_individual_files__part_6__last_ |
kliem commented
comment:6
What about
+ depends = [SAGE_INC + "/polybori/" + hd + ".h" for hd in ["polybori", "config"]]),Is that going to stay there? Is this being removed in a different ticket?
mkoeppe commented
comment:7
Ah right, that's a loose end...
kliem commented
Reviewer: Jonathan Kliem
kliem commented
comment:8
LGTM.
mkoeppe commented
comment:9
Thanks!
vbraun commented