Skip to content

board list command fails completely if even a single discovery can't be run #1666

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

Closed
silvanocerza opened this issue Feb 16, 2022 · 1 comment · Fixed by #1668
Closed

board list command fails completely if even a single discovery can't be run #1666

silvanocerza opened this issue Feb 16, 2022 · 1 comment · Fixed by #1668
Assignees
Labels
conclusion: resolved Issue was resolved topic: code Related to content of the project itself type: enhancement Proposed improvement

Comments

@silvanocerza
Copy link
Contributor

silvanocerza commented Feb 16, 2022

Bug Report

Current behavior

$ arduino-cli board list
Error detecting boards: Error getting board list: [listing ports from discovery builtin:mdns-discovery: command failed: mdns lookup error: write udp6 [::]:63214->[ff02::fb]:5353: sendto: no route to host]

The board list command runs all the discoveries installed by default, if at least one of those returns an error the command fails even if partial results could be returned.

Expected behavior

$ arduino-cli board list
Warning: `mdn-discovery` failed to run: <detail error here>
Port          Protocol Type              Board Name      FQBN                                  Core
/dev/ttyACM0  serial   Unknown
/dev/ttyACM1  serial   Serial Port (USB) Arduino MKR1000 arduino-beta-development:samd:mkr1000 arduino-beta-development:samd
                       Serial Port (USB) Arduino MKR1000 arduino:samd:mkr1000                  arduino:samd

The board list command runs all the discoveries and returns errors only for those that are failing, results from those not failing are returned.

Environment

  • CLI version (output of arduino-cli version): 0.21.0
  • OS and platform: *

Additional context

This partially handles #1543 but doesn't solve it in full.

@umbynos
Copy link
Contributor

umbynos commented Feb 3, 2023

@ubidefeo tested this. Should be solved now:

Error starting discovery: discovery builtin:mdns-discovery process not started: fork/exec /Users/ubidefeo/Library/Arduino15/packages/builtin/tools/mdns-discovery/1.0.6/mdns-discovery: no such file or directory
Port                            Protocol Type              Board Name                  FQBN                                Core
/dev/cu.Bluetooth-Incoming-Port serial   Serial Port       Unknown
/dev/cu.usbmodem101             serial   Serial Port (USB) Arduino Nano RP2040 Connect arduino:mbed_nano:nanorp2040connect arduino:mbed_nano

@umbynos umbynos closed this as completed Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: resolved Issue was resolved topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants