search

rafsaf / minimal-fastapi-postgres-template

minimal-fastapi-postgres-template based on official template but rewritten
https://minimal-fastapi-postgres-template.rafsaf.pl
MIT License
456 stars 60 forks source link

Added greenlet platform_machine arm64 to also include mac m1 #42

Closed renja-g closed 5 months ago

rafsaf commented 6 months ago

Hi, I think support is already there. I mean platform_machine == "aarch64" aarch64 == arm64 (https://stackoverflow.com/questions/31851611/differences-between-arm64-and-aarch64)

Looking here: https://pypi.org/project/greenlet/3.0.1/#files at available wheels it's ok?

Beside the above, https://python-poetry.org/docs/cli/#export poetry export was used to generate this .txt file, so after another rerun it will be overwritten :d and for all I know we can be pretty sure that poetry got it right, if not, this is greenlet related or poetry related and should be fixed there :P

PS. hmm looking more closely at https://pypi.org/project/greenlet/3.0.1/#files, indeed in requirements there is no macos platform and there is wheel greenlet-3.0.1-cp312-cp312-macosx_10_9_universal2.whl, I'm not sure if aarch64 alone covers that

rafsaf commented 5 months ago

in current version of template i deleted requirements.txt file