search

cat-in-136 / ws2812-esp32-rmt-driver

WS2812 driver using ESP32 RMT for Rust
MIT License
40 stars 22 forks source link

Upgrade the esp-idf-sys dependency to 0.32 and fix code that broke #23

Closed faern closed 1 year ago

faern commented 1 year ago

The previous version requirement of >=0.30 does not work since esp-idf-sys is allowed to change the public API in 0.31 and later, which they did. Basically changing a bunch of u32s into usize.

This PR makes the >= version requirement into a more strict caret requirement allowing only 0.32.x where x can be anything, but not 0.33 or newer. Since we can expect esp-idf-sys to not break semver this should never break our crate here.

cat-in-136 commented 1 year ago

Thank you but it is duplicated of #22 which has been merged. So I closed this PR.