-
-
Notifications
You must be signed in to change notification settings - Fork 398
stm32: Error during Upload: Property 'upload.tool.serial' is undefined #1444
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Thanks @matthijskooijman and @cmaglie |
@cmaglie @fpistm Can either of you help me figure out how to edit this? I'm struggling at finding the right files to edit. Thanks! What is editing core? And what should I define those values as (eg: what should I set |
@maxdignan we should restrict the discussion here to the Arduino CLI code base. Please post support questions to the Arduino forum: I'm sure we'll be able to help you out over there. |
The same bug affects the Real life example:
|
…it (#1629) * Factored property composition in convertUploadToolsToPluggableDiscovery * Auto-generated properties are cached and added after cycling on the original map * Autogenerated upload tool properties are searched in sub-configs too Fix #1444 * Added STMicroelectronics:stm32:Nucleo_32:pnum=NUCLEO_F031K6 to upload mock tests * Use composed board properties to detect user fields Some platforms may add information through the optional config part of the FQBN (platforms menu items).
The fix will be released with arduino-cli >=0.21.0 (or you can try the nightly as soon as it updates) |
Error during Upload: Property 'upload.tool.serial' is undefined Issue present for arduino-cli version less than 0.21.0 See arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
I had the similar error for SeeedStudio XIAO BLE. Are there any helps?
|
@cjgu00 please post over on the Arduino forum. I'll help you out over there: https://forum.arduino.cc/c/using-arduino/installation-troubleshooting/18 |
I fixed my bug by following the steps described by ptillisch, https://forum.arduino.cc/t/arduino-2-beta-12-stlink-programmer-error/921351/6 It do work! Thanks very much to give me the hint to find information on the forum. |
This reverts partially commit a4e4377. Currently, defining a default upload tools prevents flashing with arduino-cli as default methods is always used even if new one is passed as fqbn parameters. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
new version 1.2.1 raised an error about default upload method. But it could not be fixed easily as Arduino seems not manage submenu level for upload method. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
new version 1.2.1 raised an error about default upload method. But it could not be fixed easily as Arduino seems not manage submenu level for upload method. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
This reverts partially commit a4e4377. Currently, defining a default upload tools prevents flashing with arduino-cli as default methods is always used even if new one is passed as fqbn parameters. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
new version 1.2.1 raised an error about default upload method. But it could not be fixed easily as Arduino seems not manage submenu level for upload method. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
Originally posted by @matthijskooijman in #1333 (comment)
I noticed a regression that I expect is caused by this PR. When using arduino-cli from git now, uploads using (our slightly customized version of) the STM32 core fail with:
The board in question indeed does not define
upload.tool.serial
, but still uses the pre-pluggable syntax of definingupload.tool
, which should be translated toupload.tool.default
byconvertUploadToolsToPluggableDiscovery()
. However, this core does not defineupload.tool
on the board directly, but defines it in a board menu: https://github.com/stm32duino/Arduino_Core_STM32/blob/974b35690cbf4680ce5746c61c69fb20935bae2f/boards.txt#L522To reproduce:
I guess this is fairly easy to fix in the core, though (just define both
upload.tool.default
(orupload.tool.<protocol>
) in addition to the legacyupload.tool
, but it might be useful to fix compatibility here too. An obvious fix would be to implement the fallback when actually doing the upload, rather than when loading the boards).@fpistm, you might find this interesting too :-)
The text was updated successfully, but these errors were encountered: