search

Rahix / avr-hal

embedded-hal abstractions for AVR microcontrollers
Apache License 2.0
1.23k stars 216 forks source link

ravedude: Use a different avrdude config for avrdude 7.X #508

Closed Rahix closed 4 months ago

Rahix commented 4 months ago

The config we've been shipping so far was only for avrdude version 6.X. This config does not work when being used with avrdude 7.X versions.

As we do not have control over the installed avrdude version at this time, we need to provide different configurations depending on the avrdude version we find.

As a stop-gap solution, import the released default config for avrdude 7.3 and attempt using it for all avrdude 7.X versions.

In the future, we'll have to rethink this. It is probably a much better solution to simply vendor avrdude entirely.

Fixes #498.

Rahix commented 4 months ago

Please give this a try, to see whether it fixes the problem. You can install the patched ravedude like this:

cargo install --git https://github.com/rahix/avr-hal --rev 8137707b8783d2bee77f2cec4ff82e4dc0fd02e9 ravedude

Paging @weiying-chen, @Johnwf01john, and @vs-mark-graham-smith.

Johnwf01john commented 4 months ago

checked this and working OK now. Thanks

vs-mark-graham-smith commented 4 months ago

@Rahix

Checked this now and also working.

Cheers

SushantChandla commented 4 months ago

Thanks 🙏

Rahix commented 4 months ago

(Released ravedude 0.1.7 with this change)