-
-
Notifications
You must be signed in to change notification settings - Fork 402
Add --all flag to core list command and gRPC interface #1166
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
Conversation
b4e232c
to
cfe9081
Compare
cfe9081
to
a86500d
Compare
Rebased the PR, the original commit has been modified quite a bit. |
0d3124b
to
f03c76c
Compare
Setting that flags return all installed and installable platforms, including installed manually by the user in their Sketchbook hardware folder.
f03c76c
to
a71b16c
Compare
Unfortunately, it does not fix the issues, and the new gRPC API is insufficient to solve arduino/arduino-pro-ide#355. Empty state:
CLI version:
List the platforms. It correctly empty:
List all the platforms including the installable ones and the ones that were automatically installed:
Now search for a board, for example,
Install a platform manually:
Now verify the new
Now run a core search for
It does not work. So I cannot search for boards within manually installed cores. |
I feel like we need a |
Yes, that would be the ultimate solution. I am looking forward to such an API. We have to keep in mind that we need the ID of the platform for each board. This would be required when the user selects a board that is not yet installed. We need to propose to install the platform. (I know I can derive the ID of the platform from the FQBN, but I would like to avoid introducing some logic based on assumption and leave this to the CLI.) |
I'm fine with a However, I'm wondering what Akos's exact requirements are for it. Arduino Pro IDE needs to know about two classes of boards, which come from different sources:
Currently, |
This would be the user requirement from the new IDE: list (and search) for all boards that come from:
The last requirement is to open the new IDE on a brand new machine, browse a board from the dialog, pick it, and install the corresponding platform if required. |
I think it would be good to design the I think there are two distinct use cases for the command line users of
I'm thinking the default command should only show installed boards, then something like |
A |
Please check if the PR fulfills these requirements
before creating one)
our contributing guidelines
UPGRADING.md
has been updated with a migration guide (for breaking changes)Adds a new feature.
Calling
core list
returns only platforms installed with the CLI.Calling
core list
keeps returning only platforms installed with the CLI, but the--all
flag can be used to return all installed and installable platforms, including those installed manually by the user in their Sketchbook hardware folder.If both
--all
and--update
flags are set--all
takes precedence.This also updates the gRPC interface.
Nope.
Fixes arduino/arduino-pro-ide#355 and supersedes #1151
See how to contribute