Skip to content

Commit

Permalink
Specify platform tag for wheel;
Browse files Browse the repository at this point in the history
  • Loading branch information
sommerlukas committed Apr 21, 2021
1 parent 515e46d commit dc92440
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 12 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/release-build-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ jobs:
run: |
cd spn-compiler/python-interface
source $GITHUB_WORKSPACE/python/venv/bin/activate
python setup.py bdist_wheel
python setup.py bdist_wheel --plat-name linux_x86_64
- name: Upload release
uses: "marvinpinto/action-automatic-releases@latest"
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/weekly-build-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ jobs:
run: |
cd spn-compiler/python-interface
source $GITHUB_WORKSPACE/python/venv/bin/activate
python setup.py bdist_wheel
python setup.py bdist_wheel --plat-name linux_x86_64
- name: Upload release
uses: "marvinpinto/action-automatic-releases@latest"
Expand Down
12 changes: 2 additions & 10 deletions python-interface/setup.py.in
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#
# Auto-generated by CMake, do not edit!
#
from setuptools import setup, Distribution, find_packages
from setuptools import setup, find_packages
import shutil
import os

Expand All @@ -20,13 +20,6 @@ if not os.path.isfile(spncpyLib):
with open("requirements.txt", "r") as rf:
requirements = rf.readlines()

# Specify that this has external modules to
# make sure that the resulting wheel is named
# with the platform name.
class BinaryDistribution(Distribution):
def has_ext_modules(foo):
return True

setup(
name="spnc",
version="0.1",
Expand All @@ -46,6 +39,5 @@ setup(
install_requires=requirements,
setup_requires=["pytest-runner"],
tests_require=["pytest"],
test_suite="test",
distclass=BinaryDistribution
test_suite="test"
)

0 comments on commit dc92440

Please sign in to comment.