Re: Ideas for the future

[Andreas Vogel]

Am 08.03.2012 15:11, schrieb Vladimir 'φ-coder/phcoder' Serbinenko:
> On 05.03.2012 16:43, Andreas Vogel wrote:
>> Am 05.03.2012 13:54, schrieb Vladimir 'φ-coder/phcoder' Serbinenko:
>>>> option with an optional argument. By this we would have had what you
>>>> preference and my way too. Regarding optional arguments the option
>>>> handling in GRUB is actually broken by design, so this is,
>>>> unfortunately, not an option.
>>> When i implemented my patches, i wanted to use both ways: a command
>>> line
>>> Again it's unfortunate that noone noticed it when it was committed and
>>> now we're stuck with it.
>> You should think about that again. GRUB is broken regarding that and it
>> should be fixed anyway. Right now it seems that there is only one
>> command which uses GRUB_ARG_OPTION_OPTIONAL so maybe it's one of the
>> last chances to have a cheap fix.
> Learn to use rgrep before making unsupported, easy to check claims:
> ./grub-core/commands/search_wrap.c:    {"set",        's',
> GRUB_ARG_OPTION_OPTIONAL,
> ./grub-core/loader/i386/bsd.c:    {"serial", 'h',
> GRUB_ARG_OPTION_OPTIONAL,
> ./grub-core/loader/i386/bsd.c:    {"serial", 'h',
> GRUB_ARG_OPTION_OPTIONAL,
>>

Yes, you're right, I missed to make 100% sure that i checked all source
files, mea culpa. That's why i wrote "It seems that there is only one...".

BTW, I spent quite some time to investigate this issue and tried my
best, your reaction is demotivating at least. Just wanna let you know.