> -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Laszlo Ersek > Sent: Wednesday, June 5, 2019 4:03 PM > To: Ni, Ray <ray...@intel.com>; Gao, Liming <liming....@intel.com>; > ard.biesheu...@linaro.org; Leif Lindholm <leif.lindh...@linaro.org>; Kinney, > Michael D <michael.d.kin...@intel.com>; Liu, Zhiguang > <zhiguang....@intel.com>; Justen, Jordan L <jordan.l.jus...@intel.com>; > Andrew Fish <af...@apple.com> > Cc: devel@edk2.groups.io > Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > current speed > > On 06/05/19 03:10, Ni, Ray wrote: > > Hi everyone, > > > > Hao pushed the patch because: > > 1. it's a bug fix > > 2. it got a R-b. > > I don't think it's his fault. > > > > For #1, it's gap in process. > > For #2, it's my fault. Because even the patch title says EmulatorPkg but the > patch itself changes MdeModulePkg, I am still the person who can approve > changes in both packages. > > > > In Maintainers.txt: > > R: Ray Ni <ray...@intel.com> > > (especially for Bus, Universal/Console, Universal/Disk, > > Universal/BdsDxe and related libraries, header files) > > > > The UiApp actually is not in the "especially" part, but the word > > "especially" > > means I can also R-b to other components. In fact, I am not quite > > qualified on the other components, e.g.: UiApp. > > So I propose to assign clear reviewers for each components in > MdeModulePkg. > > What do you think? > > Fully agreed. MdeModulePkg is huge and we should have fine-grained > maintainership assignments.
I also agree. > > Thanks > Laszlo > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#41943): https://edk2.groups.io/g/devel/message/41943 Mute This Topic: https://groups.io/mt/31907735/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-