Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: arduino-cli upload failed (version started from 0.19.0 to 0.21.0)#1654

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
fpistm wants to merge1 commit intostm32duino:mainfromfpistm:upload_tools_default

Conversation

fpistm
Copy link
Member

@fpistmfpistm commentedFeb 10, 2022
edited
Loading

Error during Upload: Property 'upload.tool.serial' is undefined

Issue present for arduino-cli version started from 0.19.0 to 0.21.0.
Seearduino/arduino-cli#1444

/cc@matthijskooijman

Error during Upload: Property 'upload.tool.serial' is undefinedIssue present for arduino-cli version less than 0.21.0Seearduino/arduino-cli#1444Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistmfpistm added this to the2.3.0 milestoneFeb 10, 2022
@fpistmfpistm requested a review fromABOSTMFebruary 10, 2022 17:21
@matthijskooijman
Copy link
Contributor

I'm not entirely sure this fix is correct. I have not tested, but AFAUI this will no longer allow changing the upload method using the "menu" entries using older arduino-cli, since you now set a new-style upload tool for the board as a whole, but no (new-style) tool for the menu entries (only old-style tool lines, but those are not converted in 0.21).

IOW, I think you need lines like below (instead of the ones you added now):

Nucleo_144.menu.upload_method.MassStorage.upload.tool.default=massStorageCopyNucleo_144.menu.upload_method.swdMethod.upload.tool.default=stm32CubeProg

Also, might be good to mention the full range of affected arduino-cli versions, looks like this problem was introduced in 0.19 withpluggable discovery, it was already working before that.

@fpistm
Copy link
MemberAuthor

I'm not entirely sure this fix is correct. I have not tested, but AFAUI this will no longer allow changing the upload method using the "menu" entries using older arduino-cli, since you now set a new-style upload tool for the board as a whole, but no (new-style) tool for the menu entries (only old-style tool lines, but those are not converted in 0.21).

In fact with IDE it works but not with the cli.

IOW, I think you need lines like below (instead of the ones you added now):

Nucleo_144.menu.upload_method.MassStorage.upload.tool.default=massStorageCopyNucleo_144.menu.upload_method.swdMethod.upload.tool.default=stm32CubeProg

Unfortunately this is not recognized properly. I guess no fix can be provided for this.
So upload will simply not be compatible with arduino-cli version started from 0.19.0 to 0.21.0.

Now I wondering if we could use the pluggable discovery feature....

@fpistmfpistm added abandonedNo more work on this invalidThis doesn't seem right labelsFeb 11, 2022
@fpistmfpistm closed thisFeb 11, 2022
@fpistmfpistm changed the titlefix: arduino-cli upload failedfix: arduino-cli upload failed (version started from 0.19.0 to 0.21.0)Feb 11, 2022
fpistm added a commit to fpistm/Arduino_Core_STM32 that referenced this pull requestJul 21, 2022
This reverts partially commita4e4377.Currently, defining a default upload tools prevents flashingwith arduino-cli as default methods is always used even if newone is passed as fqbn parameters.See:stm32duino#1654arduino/arduino-cli#1444Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
fpistm added a commit to fpistm/Arduino_Core_STM32 that referenced this pull requestJul 21, 2022
new version 1.2.1 raised an error about default upload method.But it could not be fixed easily as Arduino seems not managesubmenu level for upload method.See:stm32duino#1654arduino/arduino-cli#1444Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
fpistm added a commit to fpistm/Arduino_Core_STM32 that referenced this pull requestJul 21, 2022
new version 1.2.1 raised an error about default upload method.But it could not be fixed easily as Arduino seems not managesubmenu level for upload method.See:stm32duino#1654arduino/arduino-cli#1444Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
cparata pushed a commit to cparata/Arduino_Core_STM32 that referenced this pull requestJan 31, 2023
This reverts partially commita4e4377.Currently, defining a default upload tools prevents flashingwith arduino-cli as default methods is always used even if newone is passed as fqbn parameters.See:stm32duino#1654arduino/arduino-cli#1444Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
cparata pushed a commit to cparata/Arduino_Core_STM32 that referenced this pull requestJan 31, 2023
new version 1.2.1 raised an error about default upload method.But it could not be fixed easily as Arduino seems not managesubmenu level for upload method.See:stm32duino#1654arduino/arduino-cli#1444Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@ABOSTMABOSTMABOSTM approved these changes

Assignees
No one assigned
Labels
abandonedNo more work on thisarduino compatibilityfix 🩹Bug fixinvalidThis doesn't seem right
Projects
None yet
Milestone
2.3.0
Development

Successfully merging this pull request may close these issues.

3 participants
@fpistm@matthijskooijman@ABOSTM

[8]ページ先頭

©2009-2025 Movatter.jp