ewilken / hap-rs

Rust implementation of the Apple HomeKit Accessory Protocol (HAP)
Apache License 2.0
196 stars 33 forks source link

Use macaddr rather than eui48 and bump dependencies #80

Closed simlay closed 1 year ago

simlay commented 1 year ago

I ended up on this tangent because when I ran cargo check on main, I saw this warning:

warning: the following packages contain code that will be rejected by a future version of Rust: rustc-serialize v0.3.24

I felt like moving to macaddr was better than trying to fix eui48 given how old https://github.com/abaumhauer/eui48/pull/34 is.

Given that part of this was to satisfy cargo audit, I decided to bump a few dependencies.

simlay commented 1 year ago

Looks like the cargo fmt --check will fail in CI. @ewilken thoughts?

simlay commented 1 year ago

Did you test the pairing process with an actual iOS controller after this change?

I just did the fan example and it worked. Any idea how this could be done in CI?