Merge pull request #121 from brunoenten/master

Working cibuildwheel github action

Author: brunoenten

.github/workflows/main.yml

@@ -0,0 +1,110 @@
+name: Build sdist and wheel and publish to PyPI and TestPyPI
+
+#on: [push, pull_request]
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  build_sdist:
+   name: Build sdist
+   runs-on: ubuntu-latest
+   steps:
+
+   - uses: actions/checkout@v3
+     with:
+       submodules: true
+
+   - name: Install ninja
+     run: pipx install ninja
+
+   - name: Build sdist
+     run: pipx run build --sdist
+
+   - name: Check metadata
+     run: pipx run twine check --strict dist/*
+
+   - uses: actions/upload-artifact@v3
+     with:
+       path: dist/*.tar.gz
+
+  build_wheels:
+    name: Build wheels on ${{ matrix.os }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
+    steps:
+
+    - uses: actions/checkout@v3
+      with:
+        submodules: true
+
+    - name: Build wheels
+      uses: pypa/cibuildwheel@v2.11.4
+
+    - uses: actions/upload-artifact@v3
+      with:
+        path: wheelhouse/*.whl
+
+  build_arch_wheels:
+   name: Build wheels on Linux ${{ matrix.arch }}
+   runs-on: ubuntu-20.04
+   strategy:
+     matrix:
+       arch: [aarch64]
+   steps:
+
+   - uses: actions/checkout@v3
+     with:
+       submodules: true
+
+   - uses: docker/setup-qemu-action@v2
+     with:
+       platforms: all
+
+   - uses: pypa/cibuildwheel@v2.11.3
+     env:
+       CIBW_ARCHS: ${{ matrix.arch }}
+
+   - name: Verify clean directory
+     run: git diff --exit-code
+     shell: bash
+
+   - name: Upload wheels
+     uses: actions/upload-artifact@v3
+     with:
+       path: wheelhouse/*.whl
+
+  upload_pypi:
+   needs: [build_arch_wheels, build_wheels, build_sdist]
+   runs-on: ubuntu-latest
+   environment: PyPI release
+   if: github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v')
+   steps:
+
+   - uses: actions/download-artifact@v3
+     with:
+       # unpacks default artifact into dist/
+       # if `name: artifact` is omitted, the action will create extra parent dir
+       name: artifact
+       path: dist
+
+   - name: Publish distribution to Test PyPI
+     uses: pypa/gh-action-pypi-publish@v1.6.4
+     with:
+       skip_existing: true
+       user: __token__
+       password: ${{ secrets.TEST_PYPI_API_TOKEN }}
+       repository_url: https://test.pypi.org/legacy/
+
+   - name: Publish distribution to PyPI
+     if: startsWith(github.ref, 'refs/tags/v')
+     uses: pypa/gh-action-pypi-publish@v1.6.4
+     with:
+       user: __token__
+       password: ${{ secrets.PYPI_API_TOKEN }}

meson.build

@@ -6,11 +6,30 @@ project(
     default_options: [
         'warning_level=2'
     ],
-    meson_version: '>=0.60.0'
+    meson_version: '>=0.63.0'
 )
 
-# Dependencies
 cpp = meson.get_compiler('cpp')
+
+## From https://github.com/numpy/numpy/blob/main/numpy/meson.build
+# Platform detection
+if host_machine.system() == 'windows' and cpp.get_id() == 'gcc'
+  # For mingw-w64, link statically against the UCRT.
+  gcc_link_args = ['-lucrt', '-static']
+
+  add_project_link_arguments(gcc_link_args, language: ['c', 'cpp'])
+  # Force gcc to float64 long doubles for compatibility with MSVC
+  # builds, for C only.
+  add_project_arguments('-mlong-double-64', language: 'c')
+  # Make fprintf("%zd") work (see https://github.com/rgommers/scipy/issues/118)
+  add_project_arguments('-D__USE_MINGW_ANSI_STDIO=1', language: ['c', 'cpp'])
+  # Manual add of MS_WIN64 macro when not using MSVC.
+  # https://bugs.python.org/issue28267
+  add_project_arguments('-DMS_WIN64', language: ['c', 'cpp'])
+endif
+##
+
+# Dependencies
 jack2_dep = dependency('jack', version: '>=1.9.11', required: false)
 jack1_dep = dependency('jack', version: ['>=0.125.0', '<1.0'], required: false)
 alsa_dep = dependency('alsa', required: false)
@@ -20,7 +39,7 @@ coremidi_dep = dependency(
     modules: ['coreaudio', 'coremidi', 'foundation'],
     required: false
 )
-winmm_dep = dependency('winmm', required: false)
+winmm_dep = cpp.find_library('winmm', required: false)
 
 if not jack2_dep.found() and jack1_dep.found()
     jack_dep = jack1_dep

pyproject.toml

@@ -90,7 +90,7 @@ manylinux-aarch64-image = "manylinux_2_28"
 build = "cp3{8,9,10,11}-manylinux*"
 archs = ["auto64"]
 before-all = [
-    "yum install -y alsa-lib-devel alsa-utils",
+    "dnf -y install alsa-lib-devel alsa-utils",
     "pipx install ninja",
     "curl -o jack2-1.9.21.tar.gz https://codeload.github.com/jackaudio/jack2/tar.gz/refs/tags/v1.9.21",
     "tar -xzf jack2-1.9.21.tar.gz",

rtmidi/meson.build

@@ -25,7 +25,9 @@ if host_machine.system() == 'linux' and alsa_dep.found() and get_option('alsa')
 endif
 
 if host_machine.system() == 'windows' and get_option('winmm')
-    defines += ['-D__WINDOWS_MM__', '/EHsc']
+    if meson.get_compiler('cpp').get_id() != 'gcc'
+        defines += ['-D__WINDOWS_MM__', '/EHsc']
+    endif
     dependencies += [winmm_dep]
 endif