[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Bug in grub-menulst2cfg
From: |
Andrei Borzenkov |
Subject: |
Re: Bug in grub-menulst2cfg |
Date: |
Thu, 22 Sep 2016 21:52:50 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 |
22.09.2016 21:08, Norman Shulman пишет:
> You're right; this patch only works for a single fallback entry, which is
> better than nothing.
>
Not really. If grub is currently booted using fallback entry, command
should fail, not set the same entry again.
> If you think nothing is better, then eliminating the first if-statement
> leaves a patch that at least fixes the NUM case.
>
This will be wrong in case of "fallback" then.
> A working NUM case makes it possible to work around the inability to save
> fallback. The example in "4.3.2 Booting fallback systems" can be rewritten
> as
>
> title A
> ...
> savedefault 1
>
> title B
> ...
> savedefault 2
>
> which is functionally equivalent.
>
> I'll think about how to handle a fallback list, but it seems hard.
>
>
> On Wed, Sep 21, 2016 at 11:31 PM, Andrei Borzenkov <address@hidden>
> wrote:
>
>> 21.09.2016 23:23, Norman Shulman пишет:
>>> I don't see how to open a bug in Bug Tracker, so I'm sending it to the
>> list.
>>>
>>> Legacy savedefault saves the current entry as the default boot entry if
>> no
>>> argument is specified. The translation only saves the first word of a
>>> multiword title.
>>>
>>> In addition, savedefault NUM and savedefault fallback are not supported.
>>>
>>> The attached patch to the latest git source fixes this.
>>>
>>
>> $fallback can be list of entries, while $saved_entry is expected to be
>> single menu entry. GRUB legacy documentation is unclear what "next
>> fallback" means here and what happens if you do several consecutive
>> "savedefault fallback" invocations, but it still explicitly says "next
>> fallback entry", not "list of fallback entries".
>>
>
>
>