Skip to content

Conversation

@silvanocerza
Copy link
Contributor

The uploader.command field in the generated module_firmware_index.json has been changed to handle some cases like different executable names for different OSs.

"uploader.command": "\"{uploader}\" -i -d --port={serial.port.file} -U true -i -e -w -v \"{loader.sketch}.bin\" -R"

becomes

"uploader.command": {
    "linux": "\"{tool_dir}/bossac\" -i -d --port={serial.port.file} -U true -i -e -w -v \"{loader.sketch}.bin\" -R",
    "windows": "\"{tool_dir}\\bossac.exe\" -i -d --port={serial.port.file} -U true -i -e -w -v \"{loader.sketch}.bin\" -R"
}
"uploader.command": "\"{uploader}\" \"-C{config.path}\" -v  -patmega4809 -cxplainedmini_updi -Pusb  -b115200 -e -D \"-Uflash:w:{loader.sketch}.hex:i\" \"-Ufuse2:w:0x01:m\" \"-Ufuse5:w:0xC9:m\" \"-Ufuse8:w:0x02:m\" "

becomes

"uploader.command": {
    "linux": "\"{tool_dir}/bin/avrdude\" \"-C{tool_dir}/etc/avrdude.conf\" -v  -patmega4809 -cxplainedmini_updi -Pusb  -b115200 -e -D \"-Uflash:w:{loader.sketch}.hex:i\" \"-Ufuse2:w:0x01:m\" \"-Ufuse5:w:0xC9:m\" \"-Ufuse8:w:0x02:m\" "
}

The internal data structures have been updated accordingly.

@silvanocerza silvanocerza requested a review from a team June 10, 2021 09:42
@silvanocerza silvanocerza self-assigned this Jun 10, 2021
@silvanocerza silvanocerza force-pushed the scerza/uploader-command-platform branch from 2a33e30 to c1699ec Compare June 10, 2021 09:46
@silvanocerza silvanocerza merged commit a99436b into main Jun 10, 2021
@silvanocerza silvanocerza deleted the scerza/uploader-command-platform branch June 10, 2021 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants