Skip to content

Only update /etc/default/grub in OEM kernel update instructions if 'quiet splash' are not included #21

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
wants to merge 2 commits into from

Conversation

rathboma
Copy link

@rathboma rathboma commented Feb 4, 2024

Don't update this line if quiet splash is present.

Fixes https://community.frame.work/t/amd-oem-kernel-update-procedure-breaks-hibernate-by-updating-grub-cmdline-linux-default-in-etc-default-grub/44726

This:

  1. Checks if quiet splash is included on the defaults line
  2. Only blitzes the line if they are not present
  3. Leaves the line as-is if quiet splash are present.

Why?

  • If a user has enabled hibernation, this will not remove their hibernate configs from this line and break hibernation (so long as they have quiet splash)

Improvements?
It might be good to be more sophisticated with this check and warn users if something will be overwritten.

@rathboma rathboma changed the title Only update if quiet splash are not included Only update /etc/default/grub in OEM kernel update instructions if 'quiet splash' are not included Feb 4, 2024
@ctsdownloads
Copy link
Collaborator

I am keeping this open, I think there is real merit to it. Going to try and make some time to put together some thoughts on how to be address this. Your pull request in on point for folks using hibernation, etc. I am putting a pin this, but keeping it open to circle back.

@ctsdownloads ctsdownloads added the enhancement New feature or request label Feb 16, 2024
@ctsdownloads ctsdownloads self-assigned this Feb 16, 2024
@ctsdownloads
Copy link
Collaborator

Patched added. Thanks for your help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants