search

trezor / cython-hidapi

:snake: Python wrapper for the HIDAPI
Other
287 stars 110 forks source link

Tox check with CI on Windows, macOS and Ubuntu #164

Closed Youw closed 1 year ago

Youw commented 1 year ago

Check most build configurations on Windows, macOS and Ubuntu.

Fixes: #104

Youw commented 1 year ago

One of the latest build results: https://github.com/Youw/cython-hidapi/actions/runs/6842472365

Youw commented 1 year ago

Unfortunately macOS check fails miserably, and I don't know why, as the very same steps on clean venv on my mac machine locally works just fine. Also, the logs strongly suggest not to invoke setup.py directly, but I don't know any other way to pass arguments like --with-system-hidapi

prusnak commented 1 year ago

Also, the logs strongly suggest not to invoke setup.py directly, but I don't know any other way to pass arguments like --with-system-hidapi

I pushed ad6c1727d0e971887b9533c066defff2353ab675 so now you can set options also via ENV variables.

Youw commented 1 year ago

Hacky. But effective. Will try.

Youw commented 1 year ago

A success! https://github.com/Youw/cython-hidapi/actions/runs/6842858099

Youw commented 1 year ago

NOTE: It appears GithubActions has a hard limit of 15 jobs to run by the build matrix. This one uses 14. If need more than that - would need to think of some workarounds like definigs a separate jobs or even actions.