Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 2c2b2f4

Browse files
committed
In the build, declare all (compulsory) extension modules together.
... in a single function. Splitting them over multiple classes doesn't really buy much. Also convert the LibAgg and Qhull classes to toplevel functions, as they play a role similar to add_numpy_flags.
1 parent 5495b88 commit 2c2b2f4

2 files changed

Lines changed: 140 additions & 202 deletions

File tree

setup.py

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -55,20 +55,9 @@
5555
setupext.Matplotlib(),
5656
setupext.Python(),
5757
setupext.Platform(),
58-
setupext.LibAgg(),
5958
setupext.FreeType(),
60-
setupext.FT2Font(),
61-
setupext.Qhull(),
62-
setupext.Image(),
63-
setupext.TTConv(),
64-
setupext.Path(),
65-
setupext.Contour(),
66-
setupext.QhullWrap(),
67-
setupext.Tri(),
6859
setupext.SampleData(),
6960
setupext.Tests(),
70-
setupext.BackendAgg(),
71-
setupext.BackendTkAgg(),
7261
setupext.BackendMacOSX(),
7362
]
7463

@@ -93,8 +82,11 @@ def __init__(self, dist):
9382

9483
class BuildExtraLibraries(BuildExtCommand):
9584
def finalize_options(self):
96-
self.distribution.ext_modules[:] = filter(
97-
None, (package.get_extension() for package in good_packages))
85+
self.distribution.ext_modules[:] = [
86+
ext
87+
for package in good_packages
88+
for ext in package.get_extensions()
89+
]
9890
super().finalize_options()
9991

10092
def build_extensions(self):

0 commit comments

Comments
 (0)