Skip to content
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

falter-berlin-autoupdate: Add error prompt for board-json #378

Merged
merged 3 commits into from
Jun 11, 2023

Conversation

Akira25
Copy link
Member

@Akira25 Akira25 commented May 7, 2023

When there was no board-json on the firmware selector autoupdate responded with a too generic error message. Thus we get more specific here.

Fixes #376

Please cherry-pick -x to 22.03 and 21.02

@Akira25 Akira25 requested a review from Noki May 7, 2023 14:03
Copy link
Member

@Noki Noki left a comment

Choose a reason for hiding this comment

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

looks good to me

that one was too quick

@Akira25 Akira25 force-pushed the add_error_prompt_autoupdate branch from ede7457 to 36276fe Compare June 4, 2023 07:55
@Akira25
Copy link
Member Author

Akira25 commented Jun 4, 2023

@Noki I also added some commits from #328 and dropped them there.

@Akira25 Akira25 force-pushed the add_error_prompt_autoupdate branch 2 times, most recently from b3a0116 to 1c207b8 Compare June 4, 2023 08:23
Copy link
Member

@Noki Noki left a comment

Choose a reason for hiding this comment

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

looks good to me

When there was no board-json on the firmware selector autoupdate responded
with a too generic error message. Thus we get more specific here.

Fixes #376

Signed-off-by: Martin Hübner <martin.hubner@web.de>
Shellcheck found some minor issues left, that were fixed with this commit.

Signed-off-by: Martin Hübner <martin.hubner@web.de>
Add a notice that autoupdate is licensed under GPL.

Signed-off-by: Martin Hübner <martin.hubner@web.de>
@Akira25 Akira25 force-pushed the add_error_prompt_autoupdate branch from 1c207b8 to c3e52c3 Compare June 5, 2023 13:44
@Akira25
Copy link
Member Author

Akira25 commented Jun 11, 2023

Merging due to approval.

@Akira25 Akira25 merged commit 3bd2610 into master Jun 11, 2023
@Akira25
Copy link
Member Author

Akira25 commented Jun 11, 2023

cherry-picked -x to 22.03 and 23.05 branches.

@pktpls pktpls deleted the add_error_prompt_autoupdate branch June 12, 2023 20:38
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.

autoupdate: improve error handling when trying to load new firmware
2 participants