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

rollback bootloader after setting default snapshot #102

Closed
wants to merge 2 commits into from

Conversation

TobiPeterG
Copy link
Collaborator

basic attempt to fix #77

Copy link
Collaborator

@aplanas aplanas left a comment

Choose a reason for hiding this comment

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

I like the approach, but IMHO one (switch) or both commands can be integrated in the existing ones

sdbootutil Outdated Show resolved Hide resolved
sdbootutil Outdated Show resolved Hide resolved
@TobiPeterG
Copy link
Collaborator Author

This makes needs_update way more verbose, personally, I like that.

@TobiPeterG
Copy link
Collaborator Author

ping
Can I rebase for merge?

@aplanas
Copy link
Collaborator

aplanas commented Aug 2, 2024

Can I rebase for merge?

Rebase it, sure. But I will need more time to review 😞

This makes sure that the bootloader version always matches the version found on the system
@TobiPeterG
Copy link
Collaborator Author

Hey, I finally found the time to rebase, feel free to test and review :)

sdbootutil Show resolved Hide resolved
sdbootutil Show resolved Hide resolved
sdbootutil Show resolved Hide resolved
sdbootutil Show resolved Hide resolved
@aplanas
Copy link
Collaborator

aplanas commented Oct 30, 2024

Moved here: #150

@aplanas aplanas closed this Oct 30, 2024
@TobiPeterG
Copy link
Collaborator Author

Oh thank you, I'm sorry that I didn't have a look at your review yet, I'm sick for quite a while now. :(

I planned on working on sdbootutil, but I was laying in bed instead. I will be though when I'm back on full power :)

@aplanas
Copy link
Collaborator

aplanas commented Oct 30, 2024

Please, take care of yourself!!

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.

Recover old bootloader and generate new predictions when doing a rollback
2 participants