platformio / platform-atmelmegaavr

Atmel megaAVR: development platform for PlatformIO
https://registry.platformio.org/platforms/platformio/atmelmegaavr
Apache License 2.0
29 stars 22 forks source link

Fix path escaping for AVRDude config file #14

Closed maxgerhardt closed 4 years ago

maxgerhardt commented 4 years ago

As noted in https://community.platformio.org/t/uploading-to-arduino-nano-every-avrdude-cant-open-config-file/17215, when the path to the config file contains a space, e.g. when the username has a space, AVRDude can't read that path anymore. Just use the full string as a normal argument without enclosing it in quotes.

The code in platform-atmelavr is of the same type, but this one here is broken.

https://github.com/platformio/platform-atmelavr/blob/a7ca0e1c30a2dbd4ee078f8f4ee5ddd68c78cc91/builder/main.py#L197-L211

valeros commented 4 years ago

Many thanks!