search

lincomatic / open_evse

Firmware for Open EVSE
GNU General Public License v3.0
114 stars 161 forks source link

pio.ini: bump atmelavr to 3.4.0 #174

Closed kolyshkin closed 3 months ago

kolyshkin commented 3 months ago

Unfortunately, since commit 3ad2310 it is not possible to use platformio on Linux:

$ pio run
Processing openevse (platform: atmelavr@3.0.0; board: openevse; framework: arduino)
-------------------------------------------------------------------------------------------------------------------
Platform Manager: Installing atmelavr @ 3.0.0
UnknownPackageError: Could not find the package with 'atmelavr @ 3.0.0' requirements for your system 'linux_x86_64'

I think that it will also make it impossible to build via using Github Actions.

Looking into https://github.com/platformio/platform-atmelavr/releases, I've tried using 3.1.0 and 3.2.0 -- with errors similar as the one above.

Updated to 3.4.0 (which appears to be the latest 3.x version) fixes the issue.

Note I've also tried building with different versions of atmelavr (with "pio run -t clean" in between to ensure everything is being rebuilt) and the resulting firmware.hex is the same for all versions (from 3.3.0 to 5.0.0, except

Signed-off-by: Kir Kolyshkin kolyshkin@gmail.com

kolyshkin commented 3 months ago

Just noticed that https://github.com/OpenEVSE/open_evse/ master branch also uses 3.4.0:

https://github.com/OpenEVSE/open_evse/blob/5d931b6d9d9d4a72ab75b83c93ec1a5088630cf1/platformio.ini#L46

lincomatic commented 3 months ago

tha't s very strange that the versions of atmelavr are different between platforms... I build on Windows 10. I'm usually hesitant to change atmelavr versions unless absolutely necessary, but since it builds the same HEX file, I'll let this one go w/o more testing.