Skip to content

Refactor setup.py #337

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
vmuriart merged 12 commits into from
Jan 30, 2017
Merged

Refactor setup.py #337

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
67c1612
Use built-in check_output (mostly)
vmuriart Jan 12, 2017
9143b54
Rename build classes, and PLATFORM to ARCH
vmuriart Jan 27, 2017
5359098
Organize imports
vmuriart Jan 27, 2017
0078d76
Move to appropriate context
vmuriart Jan 27, 2017
8fe702a
Refactor registry search
vmuriart Jan 27, 2017
de63017
Format setup.py
vmuriart Jan 28, 2017
ee21001
Organize functions
vmuriart Jan 28, 2017
b32b4d0
Create get_source_files function
vmuriart Jan 28, 2017
f520829
No need to put `setup` in `if __name__==...`
vmuriart Jan 28, 2017
126e5f0
Add long_description
vmuriart Jan 28, 2017
ccf96cb
Clean-up getinterop
vmuriart Jan 29, 2017
6521033
Fix setup.py log messages
vmuriart Jan 29, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Create get_source_files function
  • Loading branch information
@vmuriart
vmuriart committed Jan 30, 2017
commit b32b4d000000388b9f81aaf0c04f58f1885684ee
35 changes: 19 additions & 16 deletions setup.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,24 @@ def _get_interop_filename():
return os.path.join("src", "runtime", interop_filename)


def _get_source_files():
"""Walk project and collect the files needed for ext_module"""
for ext in (".sln", ".snk", ".config"):
for path in glob.glob("*" + ext):
yield path

for root, dirnames, filenames in os.walk("src"):
for ext in (".cs", ".csproj", ".sln", ".snk", ".config", ".il",
".py", ".c", ".h", ".ico"):
for filename in fnmatch.filter(filenames, "*" + ext):
yield os.path.join(root, filename)

for root, dirnames, filenames in os.walk("tools"):
for ext in (".exe", ".py", ".c", ".h"):
for filename in fnmatch.filter(filenames, "*" + ext):
yield os.path.join(root, filename)


class BuildExtPythonnet(build_ext.build_ext):
def build_extension(self, ext):
"""Builds the .pyd file using msbuild or xbuild"""
Expand Down Expand Up @@ -333,21 +351,6 @@ def run(self):
if setupdir:
os.chdir(setupdir)

sources = []
for ext in (".sln", ".snk", ".config"):
sources.extend(glob.glob("*" + ext))

for root, dirnames, filenames in os.walk("src"):
for ext in (".cs", ".csproj", ".sln", ".snk", ".config", ".il",
".py", ".c", ".h", ".ico"):
for filename in fnmatch.filter(filenames, "*" + ext):
sources.append(os.path.join(root, filename))

for root, dirnames, filenames in os.walk("tools"):
for ext in (".exe", ".py", ".c", ".h"):
for filename in fnmatch.filter(filenames, "*" + ext):
sources.append(os.path.join(root, filename))

setup_requires = []
interop_file = _get_interop_filename()
if not os.path.exists(interop_file):
Expand Down Expand Up @@ -377,7 +380,7 @@ def run(self):
'Operating System :: MacOS :: MacOS X',
],
ext_modules=[
Extension("clr", sources=sources)
Extension("clr", sources=list(_get_source_files()))
],
data_files=[
("{install_platlib}", [
Expand Down