Hi Daniel!

On 3/3/21 8:38 PM, Daniel Kiper wrote:
>> 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.

I'm glad we were able to resolve this. I was already questioning my sanity :D.

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to