Bug 1957014 - Missing "--class fedora" argument in grub.cfg - Missing icons with grub theming
Summary: Missing "--class fedora" argument in grub.cfg - Missing icons with grub theming
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: grub2
Version: 34
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Peter Jones
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-05-04 20:46 UTC by gdepak
Modified: 2021-05-04 20:46 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: ---
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: Bug


Attachments (Terms of Use)
patch for /usr/lib/kernel/install.d/20-grub.install (723 bytes, patch)
2021-05-04 20:46 UTC, gdepak
no flags Details | Diff

Description gdepak 2021-05-04 20:46:12 UTC
Created attachment 1779528 [details]
patch for /usr/lib/kernel/install.d/20-grub.install

Description of problem:

In Fedora 34, the grub menu is generated by blscfg from the files located in /boot/loader/entries/*. I think the "--class fedora" argument is missing for any fedora kernel menu. This can be a problem for grub theming for example, as the fedora icon will not be displayed on the menu.

Adding "grub_class=fedora" in the /boot/loader/entries/*.conf files will solve this issue and icons in grub menu are visible.


Version-Release number of selected component (if applicable):
grub2 2.06


How reproducible:
apply any grub theme 


Steps to Reproduce:
1. modify grub theme (using https://github.com/vinceliuice/grub2-themes for example)
2. Reboot
3. Check grub menu icons

Actual results:
"--class fedora" is missing in grub.cfg and icons are not visible in grub menu when using a theme.


Additional info:
I known this is a really minor issue (and maybe somehow it's an normal behavior) but I attached a patch for "/usr/lib/kernel/install.d/20-grub.install" that will add the missing argument for the files /boot/loader/<kernel>.conf" when installing new kernel.


Note You need to log in before you can comment on or make changes to this bug.