Merge pull request #805 from zariiii9003/cibuildwheel

Use cibuildwheel to build and test wheels
danielhrisca · Dec 31, 2022 · 65e751e · 65e751e
2 parents 05edd68 + 88e5eed
commit 65e751e
Show file tree

Hide file tree

Showing 228 changed files with 30,639 additions and 30,613 deletions.
diff --git a/.coveragerc b/.coveragerc
@@ -2,4 +2,4 @@
 source =
     asammdf
 omit =
-    asammdf/gui/*
+    */asammdf/gui/*
diff --git a/.github/workflows/build_linux.yml b/.github/workflows/build_linux.yml
@@ -34,7 +34,7 @@ jobs:
                  sudo apt update
                  sudo apt install libegl1 libopengl0
             fi
-            python -m nuitka ./asammdf/gui/asammdfgui.py --standalone --enable-plugin=numpy --enable-plugin=pyside6 --disable-console
+            python -m nuitka ./src/asammdf/gui/asammdfgui.py --standalone --enable-plugin=numpy --enable-plugin=pyside6 --disable-console
             cp asammdfd.dist dist/${RUNNER_OS}
 
       # see: https://docs.github.com/en/actions/advanced-guides/storing-workflow-data-as-artifacts

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -9,6 +9,7 @@ jobs:
       matrix:
         os: [ubuntu-22.04, macos-latest, windows-latest]
         python-version: ["3.8", "3.9", "3.10"]
+      fail-fast: false
 
     steps:
       - uses: actions/checkout@v3

diff --git a/.github/workflows/wheels.yml b/.github/workflows/wheels.yml
@@ -3,33 +3,45 @@ name: wheels
 on: [push, pull_request]
 
 jobs:
-  test:
+  build_wheels:
+    name: Build wheels on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [macos-latest, windows-latest]
-        python-version: ["3.8", "3.9", "3.10", "3.11"]
+        os: [ubuntu-latest, macos-latest, windows-latest]
+      fail-fast: false
 
     steps:
       - uses: actions/checkout@v3
 
-      - name: Set up Python
-        uses: actions/setup-python@v4
+      # Used to host cibuildwheel
+      - uses: actions/setup-python@v4
+        with:
+          python-version: "3.10"
+
+      - name: Install cibuildwheel
+        run: python -m pip install cibuildwheel
+
+      - name: Build wheels
+        run: python -m cibuildwheel --output-dir wheelhouse
+
+      - uses: actions/upload-artifact@v3
+        with:
+          path: ./wheelhouse/*.whl
 
+  build_sdist:
+    name: Build source distribution
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+
+      - uses: actions/setup-python@v4
         with:
-          python-version: ${{ matrix.python-version }}
-
-      - name: Install packages
-        run: |
-            pip install build numpy wheel 
-            python setup.py bdist_wheel
-            python setup.py sdist
-            
-      # see: https://github.com/ymyzk/tox-gh-actions
-
-      - name: Archive dist artifacts
-        uses: actions/upload-artifact@v2
+          python-version: "3.10"
+
+      - name: Build sdist
+        run: pipx run build --sdist
+
+      - uses: actions/upload-artifact@v3
         with:
-          name: dist
-          path: dist/**
-          if-no-files-found: error
+          path: dist/*.tar.gz
diff --git a/MANIFEST.in b/MANIFEST.in
@@ -1,5 +1,5 @@
 include LICENSE
 include requirements.txt
-recursive-include asammdf *.ui
-recursive-include asammdf *.c
-recursive-include asammdf *.qrc
+recursive-include src/asammdf *.ui
+recursive-include src/asammdf *.c
+recursive-include src/asammdf *.qrc
diff --git a/asammdf.spec b/asammdf.spec
@@ -4,7 +4,7 @@ from pathlib import Path
 import site
 import sys
 
-asammdf_path = Path.cwd() / "asammdf" / "gui" / "asammdfgui.py"
+asammdf_path = Path.cwd() / "src" / "asammdf" / "gui" / "asammdfgui.py"
 
 
 block_cipher = None

diff --git a/doc/conf.py b/doc/conf.py
@@ -23,7 +23,7 @@
 # BASE_DIR = os.path.abspath("..")
 # sys.path.insert(0, BASE_DIR)
 
-with open(os.path.join("..", "asammdf", "version.py"), "r") as f:
+with open(os.path.join("..", "src", "asammdf", "version.py"), "r") as f:
     for line in f:
         if line.startswith("__version__"):
             asam_version = line.split("=")[-1].strip().strip("'")

diff --git a/pyproject.toml b/pyproject.toml
@@ -4,11 +4,18 @@ build-backend = "setuptools.build_meta"
 
 [tool.black]
 extend-exclude = '''
-^/asammdf/gui/ui
+^/src/asammdf/gui/ui
 '''
 
 [tool.isort]
 force_sort_within_sections = true
 order_by_type = false
 profile = "black"
-skip_glob = ["asammdf/gui/ui"]
+skip_glob = ["src/asammdf/gui/ui"]
+
+[tool.cibuildwheel]
+test-requires = "pytest"
+test-command = "pytest {project}/test"
+build-frontend = "build"
+archs = ["auto64"]  # only build for 64bit architectures
+skip = "pp* *_ppc64le *_s390x *-musllinux*"  # skip pypy and irrelevant architectures
diff --git a/setup.py b/setup.py
@@ -15,7 +15,7 @@
 
 
 def _get_version():
-    with PROJECT_PATH.joinpath("asammdf", "version.py").open() as f:
+    with PROJECT_PATH.joinpath("src", "asammdf", "version.py").open() as f:
         line = next(line for line in f if line.startswith("__version__"))
 
     version = line.partition("=")[2].strip()[1:-1]
@@ -33,7 +33,7 @@ def _get_ext_modules():
     modules = [
         Extension(
             "asammdf.blocks.cutils",
-            ["asammdf/blocks/cutils.c"],
+            ["src/asammdf/blocks/cutils.c"],
             include_dirs=[get_include()],
         )
     ]
@@ -77,11 +77,14 @@ def _get_ext_modules():
         "Programming Language :: Python :: 3.10",
         "Programming Language :: Python :: 3.11",
     ],
+    # Supported python versions
+    python_requires=">=3.8",
     # What does your project relate to?
     keywords="read reader edit editor parse parser asam mdf measurement",
     # You can just specify the packages manually here if your project is
     # simple. Or you can use find_packages().
-    packages=find_packages(include=("asammdf*",)),
+    packages=find_packages("src"),
+    package_dir={"": "src"},
     # Alternatively, if you want to distribute just a my_module.py, uncomment
     # this:
     #   py_modules=["my_module"],
@@ -118,7 +121,7 @@ def _get_ext_modules():
     # If there are data files included in your packages that need to be
     # installed, specify them here.  If using Python 2.6 or less, then these
     # have to be included in MANIFEST.in as well.
-    package_data={"asammdf": ["asammdf/gui/ui/*.ui"]},
+    package_data={"asammdf.gui.ui": ["*.ui"]},
     include_package_data=True,
     # Although 'package_data' is the preferred approach, in some case you may
     # need to place data files outside of your packages. See:

diff --git a/asammdf/__init__.py → src/asammdf/__init__.py b/asammdf/__init__.py → src/asammdf/__init__.py
diff --git a/asammdf/blocks/__init__.py → src/asammdf/blocks/__init__.py b/asammdf/blocks/__init__.py → src/asammdf/blocks/__init__.py
diff --git a/asammdf/blocks/bus_logging_utils.py → src/asammdf/blocks/bus_logging_utils.py b/asammdf/blocks/bus_logging_utils.py → src/asammdf/blocks/bus_logging_utils.py
diff --git a/asammdf/blocks/conversion_utils.py → src/asammdf/blocks/conversion_utils.py b/asammdf/blocks/conversion_utils.py → src/asammdf/blocks/conversion_utils.py