piwheels / packages

Issue tracker for piwheels package issues
https://github.com/piwheels/packages/issues
20 stars 5 forks source link

Package issue: watchfiles #304

Closed samuelcolvin closed 1 year ago

samuelcolvin commented 2 years ago

Package name

watchfiles

Package version

all

PyPI URL

https://pypi.org/project/watchfiles/

piwheels URL

https://www.piwheels.org/project/watchfiles/

Python version

I am the maintainer

More information

See https://github.com/samuelcolvin/watchfiles/issues/149.

Watchfiles requires a newer version of rust to compile.

Any version of rust which allows edition = "2021" should work, e.g. >=1.56.0. Confirmed in https://github.com/samuelcolvin/watchfiles/pull/150.

Would it be possible to install latest stable rust via rustup for builds of this package?


Details of the error from the build logs https://www.piwheels.org/logs/0000/0825/2816.txt:

2022-05-18T01:12:30,641   Caused by:
2022-05-18T01:12:30,642     failed to parse the `edition` key

2022-05-18T01:12:30,642   Caused by:
2022-05-18T01:12:30,643     this version of Cargo is older than the `2021` edition, and only supports `2015` and `2018` editions.

I assume this build is using rust around v1.41 installed via apt or similar, but I can't confirm as I don't have a pi with me right now and the logs don't see to include the rust version.

samuelcolvin commented 2 years ago

Any update on this?

Watchfiles has binaries for armv7l as of v0.15a1, which are confirmed to work on raspberry pi, https://github.com/samuelcolvin/watchfiles/issues/149#issuecomment-1146638752.

If piwheels isn't going to provide binaries, could you just proxy/redirect to pypi?

As of this latest version, watchfiles build will require maturin, see #254.

samuelcolvin commented 1 year ago

Should be fixed when https://github.com/piwheels/piwheels/pull/322 is released.

bennuttall commented 1 year ago

Builds succeeded