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

dom0 /etc/default/grub value incorrect #4801

Closed
brendanhoar opened this issue Feb 7, 2019 · 3 comments
Closed

dom0 /etc/default/grub value incorrect #4801

brendanhoar opened this issue Feb 7, 2019 · 3 comments
Labels
C: other P: default Priority: default. Default priority for new issues, to be replaced given sufficient information.

Comments

@brendanhoar
Copy link

brendanhoar commented Feb 7, 2019

Qubes OS version:

R4.0, currently updated to testing

Affected component(s):

/etc/default/grub contents


Steps to reproduce the behavior:

Change GRUB_DISABLE_SUBMENU to the opposite value
Rebuild grub config

Expected behavior:

Grub menu changes

Actual behavior:

No change to grub menu on boot.

General notes:

In default setup, /etc/default/grub has a line like this:
GRUB_DISABLE_SUBMENU=true

However, if the intent was really to disable the submenu and show everything in a single list, it really should have been:

GRUB_DISABLE_SUBMENU="y"

After making that change and rebuilding grub config, all permutations of installed XEN and KERNELS are presented.


Related issues:

@andrewdavidwong andrewdavidwong added this to the Release 4.0 updates milestone Feb 8, 2019
@andrewdavidwong andrewdavidwong added the P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. label Mar 27, 2019
@brendanhoar
Copy link
Author

Just a reminder - really setting this correctly (as the default) might really help folks who have a broken init.rd on their newly downloaded kernel get back to a working system faster. Just noticed a confused discussion on the list today regarding something similar.

@marmarek
Copy link
Member

I see there is an upstream patch to accept all three "true", "yes" and "y".
But nevertheless, our intention was not to enable it by default, so I'll also change default value (will affect only new installs, sadly).

@brendanhoar
Copy link
Author

Oh well, that backfired. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: other P: default Priority: default. Default priority for new issues, to be replaced given sufficient information.
Projects
None yet
Development

No branches or pull requests

3 participants