Skip to content

Fix core name not showing if installed.json is not found #1230

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

Merged
merged 1 commit into from
Mar 22, 2021

Conversation

silvanocerza
Copy link
Contributor

Please check if the PR fulfills these requirements

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • UPGRADING.md has been updated with a migration guide (for breaking changes)
  • What kind of change does this PR introduce?

Fixes a bug when loading installed cores.

  • What is the current behavior?

core list doesn't return a core's name if that core folder doesn't contain an installed.json file and has been installed via CLI. Manually installed cores have their name correctly shown.

This also affects the gRPC interface.

  • What is the new behavior?

core list now returns all cores' names even if installed.json file is not found. This also fixes the gRPC interface.

Nope.

  • Other information:

It might happen that platform.txt and installed.json have different names for the same platform, thus it's not guaranted the name will be the same if those two files are not aligned.
Fixing those platforms falls on their maintainers.


See how to contribute

@silvanocerza silvanocerza requested a review from a team March 22, 2021 15:35
@silvanocerza silvanocerza self-assigned this Mar 22, 2021
Copy link

@ubidefeo ubidefeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR resolves the issue
thank you

@silvanocerza silvanocerza merged commit 0ddb024 into master Mar 22, 2021
@silvanocerza silvanocerza deleted the scerza/nameless-cores branch March 22, 2021 16:20
kittaakos pushed a commit to arduino/arduino-ide that referenced this pull request Mar 24, 2021
 - New CLI version: `20210324`.
 - The CLI has the fix for the missing core settings.

Ref: arduino/arduino-cli#1230

Closes #245.

Signed-off-by: Akos Kitta <[email protected]>
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