[edk2-devel] [EXTERNAL] Re: [PATCH v9 00/13] Add the VariablePolicy feature

[Ard Biesheuvel]

On 11/19/20 5:26 PM, Bret Barkelew wrote:
> I should clarify that it wasn’t an official process. It was just a 
> suggestion that reached some consensus.
> 

Fair enough. I haven't been as active on the mailing list recently, so I 
may have missed some of this.

I was just slightly shocked that every single platform got broken by 
this change.


> ------------------------------------------------------------------------
> *From:* devel at edk2.groups.io <devel at edk2.groups.io> on behalf of Bret 
> Barkelew via groups.io <bret.barkelew=microsoft.com at groups.io>
> *Sent:* Thursday, November 19, 2020 8:23:45 AM
> *To:* Ard Biesheuvel <ard.biesheuvel at arm.com>; Bret Barkelew 
> <debtech at gmail.com>
> *Cc:* Bret Barkelew <bret at corthon.com>; devel at edk2.groups.io 
> <devel at edk2.groups.io>; Yao, Jiewen <jiewen.yao at intel.com>; Dandan Bi 
> <dandan.bi at intel.com>; Jian J Wang <jian.j.wang at intel.com>; Hao A Wu 
> <hao.a.wu at intel.com>; Liming Gao <gaoliming at byosoft.com.cn>; Jordan 
> Justen <jordan.l.justen at intel.com>; Laszlo Ersek <lersek at redhat.com>; 
> Andrew Fish <afish at apple.com>; Ni, Ray <ray.ni at intel.com>
> *Subject:* Re: [edk2-devel] [EXTERNAL] Re: [PATCH v9 00/13] Add the 
> VariablePolicy feature
> I followed the process that was agreed upon in this list: I opened bugs 
> to every platform maintainer with explicit details of what needed to be 
> changed and waited a full month before making any further progress with 
> the main patches (adding a month to the completion time).
> 
> I’m wrapping things up to take a vacation next week, but if there are 
> still problems when I return, I can probably contribute to one or two 
> platforms.
> 
> - Bret
> ------------------------------------------------------------------------
> *From:* Ard Biesheuvel <ard.biesheuvel at arm.com>
> *Sent:* Thursday, November 19, 2020 8:19:37 AM
> *To:* Bret Barkelew <debtech at gmail.com>
> *Cc:* Bret Barkelew <bret at corthon.com>; devel at edk2.groups.io 
> <devel at edk2.groups.io>; Yao, Jiewen <jiewen.yao at intel.com>; Dandan Bi 
> <dandan.bi at intel.com>; Jian J Wang <jian.j.wang at intel.com>; Hao A Wu 
> <hao.a.wu at intel.com>; Liming Gao <gaoliming at byosoft.com.cn>; Jordan 
> Justen <jordan.l.justen at intel.com>; Laszlo Ersek <lersek at redhat.com>; 
> Andrew Fish <afish at apple.com>; Ni, Ray <ray.ni at intel.com>; Bret Barkelew 
> <Bret.Barkelew at microsoft.com>
> *Subject:* [EXTERNAL] Re: [PATCH v9 00/13] Add the VariablePolicy feature
> On 11/19/20 5:15 PM, Bret Barkelew wrote:
>> Those bugs and recommendations were sent out months ago. Several platforms have staged the changes already.
>> 
>> You need to add the library class to your DSC.
>> 
> 
> I know it has been painful to get these changes in, and I am glad that
> you stuck with it. But that does not make it OK to simply break every
> platform in edk2-platforms/ and not take any responsibility whatsoever
> for fixing it. If it is such a trivial fix, why didn't you fix it yourself?
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#67727): https://edk2.groups.io/g/devel/message/67727
Mute This Topic: https://groups.io/mt/78368776/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-