[Bug 995549] New: grub2: use $PRETTY_NAME instead of $NAME $VERSION for $GRUB_DISTRIBUTOR
http://bugzilla.suse.com/show_bug.cgi?id=995549 Bug ID: 995549 Summary: grub2: use $PRETTY_NAME instead of $NAME $VERSION for $GRUB_DISTRIBUTOR Classification: openSUSE Product: openSUSE Tumbleweed Version: Current Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: Bootloader Assignee: mchang@suse.com Reporter: lnussel@suse.com QA Contact: jsrain@suse.com CC: arvidjaar@gmail.com, dleuenberger@suse.com Found By: --- Blocker: --- grub2-default-distributor.patch uses NAME and VERSION from /etc/os-release to construct the GRUB_DISTRIBUTOR variable. With recent changes to /etc/os-release for TW the version now actually has the snapshot number. That number changes every day. Since the grub config isn't updated on every snapshot update the version listed in the bootloader might be outdated. I'd suggest to use PRETTY_NAME from /etc/os-release instead. btw, I think the grub_util_default_distributor function is unnecessarily complex. According to the specification (https://www.freedesktop.org/software/systemd/man/os-release.html) no shell special features are supported on /etc/os-release, so the file can be parsed directly, there is no need to spawn a shell. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=995549
Ludwig Nussel
http://bugzilla.suse.com/show_bug.cgi?id=995549
http://bugzilla.suse.com/show_bug.cgi?id=995549#c1
Michael Chang
grub2-default-distributor.patch uses NAME and VERSION from /etc/os-release to construct the GRUB_DISTRIBUTOR variable. With recent changes to /etc/os-release for TW the version now actually has the snapshot number. That number changes every day. Since the grub config isn't updated on every snapshot update the version listed in the bootloader might be outdated. I'd suggest to use PRETTY_NAME from /etc/os-release instead.
I think the text of varying numbers may also make openQA needles harder to match. A question, should SLE and openSUSE-Leap all change together to use $PRETTY_NAME? I think $PRETTY_NAME was once before considered but rejected while deciding which field to use .. (the discussion was in SLE, but may not apply for openSUSE). CC Thorsten and Josef.
btw, I think the grub_util_default_distributor function is unnecessarily complex. According to the specification (https://www.freedesktop.org/software/systemd/man/os-release.html) no shell special features are supported on /etc/os-release, so the file can be parsed directly, there is no need to spawn a shell.
Thanks for the information, yes it makes sense and I will do it. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=995549
http://bugzilla.suse.com/show_bug.cgi?id=995549#c2
--- Comment #2 from Michael Chang
http://bugzilla.suse.com/show_bug.cgi?id=995549
http://bugzilla.suse.com/show_bug.cgi?id=995549#c3
--- Comment #3 from Michael Chang
http://bugzilla.suse.com/show_bug.cgi?id=995549
http://bugzilla.suse.com/show_bug.cgi?id=995549#c4
--- Comment #4 from Michael Chang
http://bugzilla.suse.com/show_bug.cgi?id=995549
http://bugzilla.suse.com/show_bug.cgi?id=995549#c5
Michael Chang
participants (1)
-
bugzilla_noreply@novell.com