From: Brendan <2bndy5@gmail.com> Date: Thu, 20 Jun 2024 22:19:45 +0000 (-0700) Subject: don't upload bdist to pypi X-Git-Tag: 8.45.0^2 X-Git-Url: https://git.ayoreis.com/Adafruit_Blinka-hackapet.git/commitdiff_plain/252451e721b68894734a492b565728d4cb8317d0 don't upload bdist to pypi In short, bdist (binary distributions) created from a pure python project are universal. They cannot be made to use a platform-specific tag (like linux, windows, macos, etc). This changes the release CI workflow to skip creating/uploading a bdist to PyPI. This library has a number of dependencies conditionally required per *machine* attributes (like CPU type). But pip (or other python package managers) will not be aware of such conditional dependencies because a bdist do not invoke a dynamic resolution of machine-specific dependencies. Universal bdists only resolve platform-specific dependencies (python version, system OS, etc) which isn't sufficient for this library. Instead, this paradigm is better left to a sdist (source distribution) which carry dependency details per machine-specific attributes in the setup.py file. --- diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index a0eb8f5..a3cedce 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -35,5 +35,5 @@ jobs: for file in $(find -not -path "./.*" -not -path "./docs*" \( -name "*.py" -o -name "*.toml" \) ); do sed -i -e "s/0.0.0+auto.0/${{github.event.release.tag_name}}/" $file; done; - python -m build + python -m build -s twine upload dist/*