search

flipperzero-rs / flipperzero

Rust on the Flipper Zero
MIT License
518 stars 34 forks source link

Update bindings for firmware version 0.83.1 #70

Closed kitterion closed 1 year ago

kitterion commented 1 year ago

https://github.com/flipperdevices/flipperzero-firmware/releases/tag/0.83.1

From what I can tell, it's a relatively large diff because the order of a lot of structs and functions got changed without any actual changes. Seems like the order of include paths got shuffled around a bit probably because of this item in the release notes:

Yahyaaddailam commented 1 year ago

Update

kitterion commented 1 year ago

I've updated the tools/Dockerfile, it should produce byte-for-byte identical bindings.rs.

cptpiepmatz commented 1 year ago

So is this ready?

gmelodie commented 1 year ago

Any updates on this? I tried making an app with @kitterion's branch but I still get the same API mismatch error.

str4d commented 1 year ago

So is this ready?

I believe so, although I now see that upstream bumped their SDK version again yesterday with firmware version 0.84.0-rc, so the upcoming firmware release will be incompatible again. From that perspective, I'm inclined to merge this (so people can work from the main branch), but wait the extra week or so until the next official firmware release, and then push out a crate version targeting that.

Any updates on this? I tried making an app with @kitterion's branch but I still get the same API mismatch error.

@gmelodie that implies your Flipper Zero's firmware version is not either 0.82.3 (required for the version 0.9.0 crates) or 0.83.1 (required for this branch). Can you check what version you are running (either via qFlipper, or `Settings -> About -> last page)?

str4d commented 1 year ago

0.84.1 was released 2 hours ago! I'll merge this and then get another PR up for that bump.