> -----Original Message-----
> From: Leif Lindholm [mailto:l...@nuviainc.com]
> Sent: Sunday, August 30, 2020 7:44 PM
> To: Chang, Abner (HPS SW/FW Technologist) <abner.ch...@hpe.com>
> Cc: devel@edk2.groups.io; Andrew Fish <af...@apple.com>; Laszlo Ersek
> <ler...@redhat.com>; Michael D Kinney <michael.d.kin...@intel.com>;
> Leif Lindholm <leif.lindh...@linaro.org>
> Subject: Re: [PATCH 1/1] Maintainers.txt: Update reviewers of */RiscV64
>
> On Sun, Aug 30, 2020 at 16:03:06 +0800, Abner Chang wrote:
> > Add reviewers for all /RiscV64 folders.
> >
> > Signed-off-by: Abner Chang <abner.ch...@hpe.com>
> > Cc: Andrew Fish <af...@apple.com>
> > Cc: Laszlo Ersek <ler...@redhat.com>
> > Cc: Leif Lindholm <l...@nuviainc.com>
> > Cc: Michael D Kinney <michael.d.kin...@intel.com>
> > Cc: Leif Lindholm <leif.lindh...@linaro.org>
> > ---
> > Maintainers.txt | 5 +++++
> > 1 file changed, 5 insertions(+)
> >
> > diff --git a/Maintainers.txt b/Maintainers.txt index
> > 57cd2fc662..ff34a24abd 100644
> > --- a/Maintainers.txt
> > +++ b/Maintainers.txt
> > @@ -98,6 +98,11 @@ F: */Arm/
> > M: Leif Lindholm <l...@nuviainc.com>
> > M: Ard Biesheuvel <ard.biesheu...@arm.com>
> >
> > +RISCV64
> > +F: */RiscV64/
> > +R: Abner Chang <abner.ch...@hpe.com>
> > +R: Daniel Schaefer <daniel.schae...@hpe.com>
>
> I think we want an M: entry for this, and I think that should be you (since
> you
> have the access privileges). It signifies that you are the one responsible for
> patches that have been reviewed actually get pushed.
No, I don't have access to edk2 repo, will ask Mike for this.
>
> This doesn't always mean you need to do it yourself, especially for this entry
> which will always overlap with other entries (like MdePkg,
> MdeModulePkg, ...), but it does make it clear that you need to make sure it
> doesn't get dropped.
>
> With that change:
> Reviewed-by: Leif Lindholm <l...@nuviainc.com>
>
> > +
> > EDK II Continuous Integration:
> > ------------------------------
> > .azurepipelines/
> > --
> > 2.25.0
> >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#64810): https://edk2.groups.io/g/devel/message/64810
Mute This Topic: https://groups.io/mt/76510535/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
