[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [SECURITY PATCH 116/117] templates: Disable the os-prober by default

From: Daniel Kiper
Subject: Re: [SECURITY PATCH 116/117] templates: Disable the os-prober by default
Date: Wed, 3 Mar 2021 20:38:34 +0100
User-agent: NeoMutt/20170113 (1.7.2)

On Wed, Mar 03, 2021 at 07:43:23PM +0100, John Paul Adrian Glaubitz wrote:
> Hi Daniel!
> On 3/3/21 2:13 PM, Daniel Kiper wrote:
> >> This is confusing: now to get boot entries from os-prober one have to
> >> set:
> >> in /etc/default/grub.
> >>
> >> Either revert that, or (better, in my opinion) label the variable
> >> GRUB_ENABLE_OS_PROBER and set it to false by default.
> >
> > When we worked on this patch we considered that. However, after some
> > thinking we stated that renaming to GRUB_ENABLE_OS_PROBER will make
> > more confusion. So, we decided to stick to existing name even if it
> > is not the best one.
> I'm not sure I'm able to follow the motivation to keep the naming scheme
> as is.
> If I have an option called "DISABLE_FEATURE_X", then I would expect setting
> it to "true" would mean that the feature is disabled not the other way around.

Huh! I have just realized that we did not drop "if" change from earlier version
of patch which also changed variable name. Interestingly doc update is correct.
Anyway, sorry for the confusion. I asked Alex to post a fix for this.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]