[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug #63481] 30_os-prober test GRUB_OS_PROBER_SKIP_LIST wrong
From: |
David Garfield |
Subject: |
[bug #63481] 30_os-prober test GRUB_OS_PROBER_SKIP_LIST wrong |
Date: |
Wed, 7 Dec 2022 06:04:04 -0500 (EST) |
URL:
<https://savannah.gnu.org/bugs/?63481>
Summary: 30_os-prober test GRUB_OS_PROBER_SKIP_LIST wrong
Project: GNU GRUB
Submitter: divad27182
Submitted: Wed 07 Dec 2022 11:04:02 AM UTC
Category: Configuration
Severity: Major
Priority: 5 - Normal
Item Group: Software Error
Status: None
Privacy: Public
Assigned to: None
Originator Name: David Garfield
Originator Email: david@barlinq.com
Open/Closed: Open
Release: other
Release:
Discussion Lock: Any
Reproducibility: Every Time
Planned Release: None
_______________________________________________________
Follow-up Comments:
-------------------------------------------------------
Date: Wed 07 Dec 2022 11:04:02 AM UTC By: David Garfield <divad27182>
According to the documentation, the @name clause is added to the test string
for EFI chainloaders. This is based on os-prober outputting the same clause.
os-prober output data seems to make the first field be either a device name,
or a device name "@" chainloader path. Unfortunately, the test for the
presence of the @ reads:
if [ x"${DEVICE#*@}" != x ] ; then
This tests for the presence of something after the "@", but is always true if
there is no "@".
I think the test should read:
if [ x"${DEVICE#*@}" != x"${DEVICE}" ] ; then
It may be that a combined test (differentiating "A", "A@", and "A@B") may be
needed, if os-prober has changed its output format.
This is line 124 of util/grub.d/30_os-prober.in in grub-2.06
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?63481>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [bug #63481] 30_os-prober test GRUB_OS_PROBER_SKIP_LIST wrong,
David Garfield <=